Skip to content

MWPW-189475 When only 3 editorial cards authored in carousel and veri…#912

Closed
suhjainadobe wants to merge 2 commits intostagefrom
carousel-swipe
Closed

MWPW-189475 When only 3 editorial cards authored in carousel and veri…#912
suhjainadobe wants to merge 2 commits intostagefrom
carousel-swipe

Conversation

@suhjainadobe
Copy link
Contributor

[Doodlebug] When only 3 editorial cards authored in carousel and verified in ipad, functionality is not seen as expected

  • Add your specific features or fixes

Resolves: MWPW-189475

Test URLs:

…fied in ipad, functionality is not seen as expected
@aem-code-sync
Copy link

aem-code-sync bot commented Mar 7, 2026

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

@suhjainadobe suhjainadobe requested a review from nkthakur48 March 7, 2026 02:17
@aem-code-sync
Copy link

aem-code-sync bot commented Mar 7, 2026

Page Scores Audits Google
📱 /cc-shared/fragments/products/firefly/features/ai-image-expander/z-carousel?milo
libs=stage
Lighthouse returned error: NOT_HTML. The page provided is not HTML (served as MIME type text/plain). PSI
🖥️ /cc-shared/fragments/products/firefly/features/ai-image-expander/z-carousel?milo
libs=stage
Lighthouse returned error: NOT_HTML. The page provided is not HTML (served as MIME type text/plain). PSI

@spadmasa spadmasa self-assigned this Mar 7, 2026
@milo-pr-merge-cc
Copy link

Skipped merging 912: MWPW-189475 When only 3 editorial cards authored in carousel and veri… due to failing checks

}

.carousel.hinting-tablet:has(.carousel-slides.is-reversing .carousel-slide:first-child.active) .carousel-slide.reference-slide {
order: 999 !important;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are there any better ways to do it? 999 and !important are big red flags per me.

@nkthakur48
Copy link
Collaborator

nkthakur48 commented Mar 9, 2026

@drashti1712, Could you please have a look at review comment since @suhjainadobe is on leave?
CC: @skambi

@drashti1712
Copy link
Collaborator

Closing this PR as I have raised a new one: #915

@drashti1712 drashti1712 closed this Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants