-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Emulate reduced-motion in e2e tests #34132
Conversation
Size Change: +7.95 kB (+1%) Total Size: 1.04 MB
ℹ️ View Unchanged
|
@@ -240,6 +240,9 @@ beforeAll( async () => { | |||
await trashAllPosts( 'wp_block' ); | |||
await setupBrowser(); | |||
await activatePlugin( 'gutenberg-test-plugin-disables-the-css-animations' ); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this means we can remove that plugin no?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, but I'd rather keep it for a while though. As mentioned in the original issue, I don't think we're quite there yet to disable this plugin entirely. Disabling it now would probably slow down the tests a little too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice one 👍
process.env.FORCE_REDUCED_MOTION || IS_IE | ||
? () => true | ||
: () => useMediaQuery( '(prefers-reduced-motion: reduce)' ); | ||
const useReducedMotion = IS_IE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can remove the is_ie check now :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice one
Description
Originally brought up in #33317 (comment). Emulate
prefers-reduced-motion: reduce
in e2e tests.How has this been tested?
E2E tests should pass
Types of changes
New feature
Checklist:
*.native.js
files for terms that need renaming or removal).