components
components copied to clipboard
chore: Remove unnecessary class to disable animations in tests
Description
Since we use the option prefers-reduced-motion
at the browser level for testing, there is no need to animations and transitions via CSS as well, with the exception of the spinner component, which by design has animations even when reduced motion is set.
Document: HViwARxNHC6F
How has this been tested?
Existing integration tests.
Review checklist
The following items are to be evaluated by the author(s) and the reviewer(s).
Correctness
- Changes include appropriate documentation updates.
-
Changes are backward-compatible if not indicated, see
CONTRIBUTING.md
. -
Changes do not include unsupported browser features, see
CONTRIBUTING.md
. - Changes were manually tested for accessibility, see accessibility guidelines.
Security
-
If the code handles URLs: all URLs are validated through the
checkSafeUrl
function.
Testing
- Changes are covered with new/existing unit tests?
- Changes are covered with new/existing integration tests?
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
6a1404a
) 94.63% compared to head (7138a94
) 94.63%.
Additional details and impacted files
@@ Coverage Diff @@
## main #1822 +/- ##
=======================================
Coverage 94.63% 94.63%
=======================================
Files 654 654
Lines 17794 17794
Branches 5867 5867
=======================================
Hits 16840 16840
Misses 887 887
Partials 67 67
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.