open-props icon indicating copy to clipboard operation
open-props copied to clipboard

Issue 192

Open Que-tin opened this issue 3 years ago • 7 comments

Que-tin avatar Feb 17 '22 23:02 Que-tin

Done. Somone needs to check the reduced motion timings. Especially shake & bounce are behaving a bit weird.

Que-tin avatar Feb 23 '22 18:02 Que-tin

did some polish and fixed an animation, made a PR to your branch! https://github.com/Que-tin/open-props/pull/1

argyleink avatar Feb 24 '22 05:02 argyleink

@argyleink Checked and tested everything, looks good to me 👍🏻😊

Your PR also got merged.

Que-tin avatar Feb 24 '22 14:02 Que-tin

Next step is I'll host the feature and send it to a few folks I trust and who prefer reduced motion, have them test it and give it some QA 👍🏻

argyleink avatar Feb 24 '22 15:02 argyleink

preview URL https://op-reduced-animations.netlify.app/#animations

argyleink avatar Feb 24 '22 15:02 argyleink

review #1:

I agree but I can only speak for myself: most of the time I'm okay with small animations (like color fade), it's mostly paralax and things moving around too much that trigger me ^^ And spinning hahaha spinning is NOOOO good for my brain 😄 haa the spining even with motion reduced is a big NOPE hahaa The step by step animations feel strange without transition, to be honnest. It's a little bit hard to give you feedback since it also depends on how those are used in a page The bounce is also a big nope at the moment even in the reduce motion, but the pulse is okay. So I think it's the size of the change and the speed as well that impacts but again, it might depend on the level of sensitivity to motion sickness

argyleink avatar Feb 24 '22 15:02 argyleink

i still think about these efforts and where they should go 🙂

argyleink avatar May 12 '22 04:05 argyleink

Closed due to the progress pn #341

Que-tin avatar Apr 28 '23 18:04 Que-tin