-
Notifications
You must be signed in to change notification settings - Fork 64
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
[Accessibility] Animations in theme should use reduced motion media queries #386
[Accessibility] Animations in theme should use reduced motion media queries #386
Comments
Please don't use this via the media query alone. It should also be switched in the json. |
Please use html class |
is-prefers-reduced-motion is available in latest core |
I'm assuming |
I think the scale & pulse animations could just be done like this, although it might be worth checking what they have been used for or what intended to be used for. _onScreen I'd say maybe replace these with a fade perhaps rather than just have no animation at all. I have no idea what image zoom is! |
Image zoom is a scale effect applied to graphics on hover for a bit of pizazz e.g. menu item image. |
@kirsty-hames And we all love a bit of pizazz! I remember now.. On hover? Isn't that a little confusing though? Is it ever used? |
Yep on hover. I've seen it used on menu and hotgrid items but I think it's odd to have it on images where there isn't a click event e.g. graphic component. A hover event implies there is an action. |
Accessibility
Animations should have accompanying reduced motion media queries, or have a media query that works globally. So something along the lines of:
@media (prefers-reduced-motion: reduce) { .has-animation { animation: none !important; } }
https://developer.mozilla.org/en-US/docs/Web/CSS/Media_Queries/Using_Media_Queries_for_Accessibility
The text was updated successfully, but these errors were encountered: