-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
Add option to disable ripples globally #1776
Comments
This should probably be similar to disabling animations globally. |
It would be a nice option to have. Especially on mobile devices, the ripples animations kill anything but the top dog phones. |
@tinayuangao Seems to be
feat(ripple): support for global ripple options (#3463) |
So it is. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Bug, feature request, or proposal:
An option to globally disable ripple animations would help to reduce flakiness in screendiff testing.
What is the expected behavior?
Globally diable ripple animations for screen diff testing.
What is the current behavior?
Ripple animations can only be disabled on an element by element basis.
What are the steps to reproduce?
What is the use-case or motivation for changing an existing behavior?
Making screenshot diff tests more reliable.
Which versions of Angular, Material, OS, browsers are affected?
angular/material2
Is there anything else we should know?
The text was updated successfully, but these errors were encountered: