Skip to content

[MWPW-199058]: Fixed firefly-carousel behaviour in reduced motion.#93

Open
hkuraware wants to merge 2 commits into
stagefrom
mwpw-199058-firefly-carousel-rm
Open

[MWPW-199058]: Fixed firefly-carousel behaviour in reduced motion.#93
hkuraware wants to merge 2 commits into
stagefrom
mwpw-199058-firefly-carousel-rm

Conversation

@hkuraware

Copy link
Copy Markdown

Summary

Fixes firefly-carousel videos auto-playing when prefers-reduced-motion: reduce is enabled (e.g. Windows “Animation effects” off).

  • Disables video autoplay and pauses all carousel videos on init when reduced motion is on
  • Disables carousel autoscroll under reduced motion
  • Disables track slide transition animation via CSS
  • If a user manually plays a video, it keeps playing only while in the carousel viewport; once swiped off-screen it pauses and stays paused until the user plays it again (uses Milo USER_PAUSED_ATTR + syncPausePlayIcon)
  • No Milo decorate import or viewport checks when reduced motion is off — existing carousel behavior is unchanged

Resolves: MWPW-199058

Test URLs:

Dev validation:

Before:-
before-fix

After:-
after-fix

@aem-code-sync

aem-code-sync Bot commented Jun 22, 2026

Copy link
Copy Markdown

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 all PSI checks
  • Re-run failed PSI checks
  • Re-sync branch
Commits

@aem-code-sync

aem-code-sync Bot commented Jun 22, 2026

Copy link
Copy Markdown

@SushmaNayak630

Copy link
Copy Markdown

Hi @hkuraware,
When a user manually plays a video while it is in the viewport and then swipes it off-screen, the video continues to play instead of pausing.
image

@hkuraware

Copy link
Copy Markdown
Author

Hi @hkuraware,
When a user manually plays a video while it is in the viewport and then swipes it off-screen, the video continues to play instead of pausing.

Hi @SushmaNayak630 ,
Thanks for checking this. This change applies only when reduced motion is enabled, otherwise the carousel behaves as before.
Under reduced motion, a user-played video should pause when it leaves the viewport and not resume until the user plays it again. Could you confirm you’re testing with reduced motion on (e.g. Windows Animation effects off)?
cc: @spadmasa

@SushmaNayak630

Copy link
Copy Markdown

Hi @hkuraware,
When a user manually plays a video while it is in the viewport and then swipes it off-screen, the video continues to play instead of pausing.

Hi @SushmaNayak630 , Thanks for checking this. This change applies only when reduced motion is enabled, otherwise the carousel behaves as before. Under reduced motion, a user-played video should pause when it leaves the viewport and not resume until the user plays it again. Could you confirm you’re testing with reduced motion on (e.g. Windows Animation effects off)? cc: @spadmasa

@hkuraware I was checking when reduce motion is enabled and we were referring to the videos that are partially visible on the left and right sides of the carousel. Since they are still partially visible and therefore considered within the viewport, the behavior mentioned above is expected

@SushmaNayak630

Copy link
Copy Markdown

Validated in branch
https://mwpw-199058-firefly-carousel-rm--da-cc--adobecom.aem.live/products/firefly/features/ai-animation-generator
Disables video autoplay and pauses all carousel videos on init when reduced motion is on
image
If a user manually plays a video, it keeps playing
image
image

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.

3 participants