From 49d51f78e4681d558ae21daa080c281bd2a2d8c0 Mon Sep 17 00:00:00 2001 From: sai Date: Sat, 29 Apr 2023 14:17:52 +0200 Subject: [PATCH] Revert "chore: include color helper (#355)" This reverts commit bd493f61e06cd6df9b95b47648df448d04c96eb8. --- scratch.md | 57 ------------------------------------------------------ 1 file changed, 57 deletions(-) delete mode 100644 scratch.md diff --git a/scratch.md b/scratch.md deleted file mode 100644 index bdc3ac9f..00000000 --- a/scratch.md +++ /dev/null @@ -1,57 +0,0 @@ -# scratch - -## new feedback - -provide a codepen with them side by side, kinda like we have now. explain how to use devtools to emulate reduced motion so they can preview the effect. - -for a detailed overview of our reduced motion animations check here - ---- - -## reply - -> Thanks for putting in the work & sorry for not finishing #194! - -@Que-tin no worries. as you can see, i haven't been moving super fast either. 🐢 thanks for the first iteration. - ---- - -> a toggle on the top right of the animation sample section to give people an option to easily inspect those reduced animations? - -i like the idea of a seperate codepen because a user can see them side by side at the same time, which isn't possible if they have to toggle between them or change browser options. - -also maybe an implementation example. - ---- - -### new reply - -whoops! i forgot that pushing would trigger the CI. sorry, my bad. - -> hope my early review didnt cramp your style - -no worries, just needed to clean up a bit. i implemented the suggestions from you and @Que-tin, and cleaned up the diff. the documentation might need some work. everything else is ready for review! - -also i tested the new animations and it looks good on my side. i used the latest stable Chrome and Firefox on Linux. maybe @Que-tin could also test it out? - ---- - -a summary: -the "attention getters" have too much movement to emulate. even a reduced version can still be disorientating when played too fast. since the user is in control of the speed, we switch it to the blink animation, because it achieves the same intented purpose: attention getter. - -in/out that has similar reduced effect: - -- fade-{in,out} (no change) -- fade-{in,out}-bloom (no change) -- scale-{up,down} -- slide-{in,out}-{up,down,left,right} - -"attention getters" (start and end are the same) reduced to blink: - -- blink (no change) -- shake-{x,y} -- ping -- float -- pulse -- spin -- bounce