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
Spinner: Change spinner icon when prefers-reduced-motion
is set
#87641
Conversation
ok something weird is going on with the but that is definitely not a friday afternoon job 😂 will come back to it on monday 🤓 |
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 👍
)} | ||
name={iconName} | ||
size={size} | ||
aria-label="loading spinner" |
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.
While where here should we get rid of this aria-label? 🥺
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.
does it need replacing with anything or are we just straight removing it? 🤔
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.
Hmm... after a brief look, about 75% use the spinner without a text label (and so would require a aria-label). Maybe lets just change it to 'loading' for now?
What is this feature?
prefers-reduced-motion
is setmatchMedia
mocks as well since they weren't working correctly 🤷Why do we need this feature?
Who is this feature for?
Which issue(s) does this PR fix?:
Fixes #
Special notes for your reviewer:
Please check that: