Skip to content

🐛 [Story a11y] Reduced motion should apply the animation's last frame to the elements#34466

Merged
mszylkowski merged 67 commits intoampproject:mainfrom
mszylkowski:animation_reducedmotion
Jul 2, 2021
Merged

🐛 [Story a11y] Reduced motion should apply the animation's last frame to the elements#34466
mszylkowski merged 67 commits intoampproject:mainfrom
mszylkowski:animation_reducedmotion

Conversation

@mszylkowski
Copy link
Copy Markdown
Contributor

@mszylkowski mszylkowski commented May 20, 2021

Closes #34443

If prefers-reduced-motion, still creates the animation manager.

On maybeStartAnimations, check if prefers-reduced-motion and if so, call maybeApplyLastAnimationFrame. This calls Animation.finish() or Animation.pause (if animation is infinite).

Demo at https://stories-demos-matias.web.app/examples/amp-story/amp-story-animation.html

reduced-motion.mp4

mszylkowski and others added 30 commits April 6, 2021 12:58
@mszylkowski mszylkowski requested a review from alanorozco June 2, 2021 17:25
Copy link
Copy Markdown
Member

@alanorozco alanorozco left a comment

Choose a reason for hiding this comment

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

Thank you for your patience on the back and forth. LGTM!

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.

[Story a11y] Reduced motion breaks layouts with animate-in

5 participants