-
-
Notifications
You must be signed in to change notification settings - Fork 91
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
Accessibility improvement opportunities #745
Comments
Hello @Shane-Donlon, thanks for opening your first issue! your contribution is valuable to us. The maintainers will review this issue and provide feedback as soon as possible. |
I think I can take it, but it require design changes. Should I focus only on contrast, ore something else? Or wait for storybook (a11y tests can be done there) |
@Maczi01 - I'm looking into this! I have some new designs. I also want to see how Storybook works with severside components. I know they recently started supporting this, and that's the way we are heading. The new designs will hopefully get rid of some of these issues. Some of these things don't need to be fixed (like the CMD+K text) because it's just a placeholder and they can often fail here. The button to scroll down definitely could use an accessible name, though, if you want to take that? |
Thanks for quick answer @NiallJoeMaher ! I heard they added support for RSC, but didn't try yet. No rush! If you need, I can try to help with it. I can handle scroll issue, assign to me pls :-) |
Context
Please provide any relevant information about your setup:
Page Speed Insights results
Feedback:
Screenshots
The text was updated successfully, but these errors were encountered: