Skip to content
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.

Sign up for GitHub

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

Jump to bottom

'prefers-reduced-motion' and transition-duration: 0s #9

Open
ciar4n opened this issue Feb 18, 2021 · 0 comments
Open

'prefers-reduced-motion' and transition-duration: 0s #9

ciar4n opened this issue Feb 18, 2021 · 0 comments

Comments

@ciar4n
Copy link

ciar4n commented Feb 18, 2021

Describe the bug
Appears to be an issue if the following is part of the CSS and reduced motion is enabled on the OS...

@media (prefers-reduced-motion: reduce) { 
   *, ::before, ::after { 
     transition-duration: 0s !important; 
   } 
 }

The first event on the menu works fine but after that, nothing, the menu completely freezes up with no errors. Strangely works fine if transition-duration: 0.000000001s, just not with transition-duration: 0s

Expected behavior
Menu works fine with no animations

Desktop (please complete the following information):

  • OS: Windows 10]
  • Browser CHrome / Firefox (latest)
  • Latest
@richard67 richard67 mentioned this issue Feb 18, 2021
Closed
brianteeman added a commit to brianteeman/joomla-cms that referenced this issue Apr 8, 2021
@brianteeman
[4.0] admin menu reduced motion
ee36788 
pr for joomla#31242

This is a temp fix until upstream addresses this onokumus/metismenujs#9

The fix applied is the exact same one used by webaim.org so should be acceptable from an a11y perspective.

There will be no change for most users. It is only if they have reduced motion set in their operating system that they encounter the problem this fixes.
@brianteeman brianteeman mentioned this issue Apr 8, 2021
Merged
HLeithner pushed a commit to joomla/joomla-cms that referenced this issue Apr 8, 2021
@brianteeman
[4.0] admin menu reduced motion (#33069)
03c1bf1 
pr for #31242

This is a temp fix until upstream addresses this onokumus/metismenujs#9

The fix applied is the exact same one used by webaim.org so should be acceptable from an a11y perspective.

There will be no change for most users. It is only if they have reduced motion set in their operating system that they encounter the problem this fixes.
onokumus added a commit that referenced this issue Jan 11, 2023
@onokumus
add no-transition options
9e92ec4 
#9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ciar4n