Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Release] Sync main from stage 5/14 #296

Merged
merged 7 commits into from
May 15, 2024
Merged

[Release] Sync main from stage 5/14 #296

merged 7 commits into from
May 15, 2024

Conversation

Ruchika4 and others added 6 commits May 7, 2024 08:44
* MWPW-143894 moving category authoring to separate row

* MWPW-143894 more robust for edge cases

* draft

* simplification

* other fixes not brought in

* adding remnant stuff from MWPW-142148

* MWPW-143894 fix parent id

* changed order of interactive marquee for Mweb users

* fixed linting errors

* code enhancement

* spacing and padding adjustents for firefly marquee

* multi image firefly marquee

* css update

* handle responsive design for tablet

* MWPW-143904-New block: cc-mobile-app-banner (#271)

* app banner in cc

* app banner in CC

* add review comments

* code enhancement

wq
!
!

* Revert "MWPW-143904-New block: cc-mobile-app-banner (#271)"

This reverts commit 6c5a816.

* code enhancement

* DOTCOM-118347

* optimize css

* fix placeholder italic

* add unit tests

* tokenized the feature requested

* handle touch device

* MWPW-143894 some fixes

* wip

* wip

* Revert "DOTCOM-118347"

* add unit tests

* wip

* wip

* blinking cursor

* pull mweb changes

* increase aytocycle time

* Update milo-marquee.css

* Update firefly-masonry.css

* Optimize

* unit test

* optimize

* remove unnecessary code

* fix css

* support dark masonry promptbar

* css update

* fix intermittent CLS issue

* fix duplicate id issue for accessibility

* fix unit test

* change background img to img for SEO reasons

* incorporate review comments

* fix analytics for touch device

* css

* lint issue

* button css fix

* css fixes

* css fix

* css fix

* css fix

* css fix

* css fix

* fix foreground width

* css fix

* css fix

* remove unnecessary code

* Update firefly-masonry.css

* fix rtl

* optimize

* remove importing interactive-elements.css

* css fix

* fix table responsiveness

* fix MWPW-147643

* Update firefly-masonry.js

* Update firefly-masonry.js

* incorporate review comments

* review comments

* lint issue

* lint issue

* css fix

* set box sizing to fix isue when caas on page

* set box sizing to fix isue when caas on page

---------

Co-authored-by: Nicolas Peltier <npeltier@adobe.com>
Co-authored-by: sharath kannan <shkannan@adobe.com>
Co-authored-by: sharathkannan <138484653+sharath-kannan@users.noreply.github.com>
Co-authored-by: Aishwarya Mathuria <mathuria@adobe.com>
Co-authored-by: Blaine Gunn <Blainegunn@gmail.com>
Co-authored-by: Nicolas Peltier <1032754+npeltier@users.noreply.github.com>
Co-authored-by: Saloni Jain <6162294+salonijain3@users.noreply.github.com>
High LCP was seen for hue-sat marquee.
Resolves: MWPW-146221
* Update firefly-masonry.js

* Update firefly-masonry.js

* fix lint issues

* Update firefly-masonry.js

* Update firefly-masonry.js

* incorporate review comments
Continue to photoshop button is not seen working in stage ,proxy denied error is seen. Thus changed the dev env to stage.
Resolves: MWPW-148165
Copy link

aem-code-sync bot commented May 14, 2024

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run PSI checks
  • Re-sync branch
Commits

Copy link

aem-code-sync bot commented May 14, 2024

Page Scores Audits Google
/careers?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@Ruchika4 Ruchika4 requested review from dstrong23 and drashti1712 May 14, 2024 16:31
@aishwaryamathuria aishwaryamathuria merged commit b38a7c0 into main May 15, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants