You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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
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.
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 @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
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
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes firefly-carousel videos auto-playing when prefers-reduced-motion: reduce is enabled (e.g. Windows “Animation effects” off).
USER_PAUSED_ATTR+syncPausePlayIcon)Resolves: MWPW-199058
Test URLs:
Dev validation:
Before:-

After:-
