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.

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 notes to readme/documentation on accessibility #131

Open
jkjustjoshing opened this issue Jun 1, 2020 · 1 comment
Open

Add notes to readme/documentation on accessibility #131

jkjustjoshing opened this issue Jun 1, 2020 · 1 comment

Comments

@jkjustjoshing
Copy link

Right now there's no note on how this library acts for a screen reader, keyboard user, or just a user that doesn't read very fast. These are important considerations for a UI component to consider.

Please check out https://www.w3.org/WAI/tutorials/carousels/ for more information on things to consider.

2 specific issues I noticed with a quick glance at the docs:

  1. The buttons in the button example aren't buttons! Not friendly to keyboard OR screen reader users
  2. The link to the keyboard navigation example from the README is broken. Keyboard navigation is an important thing for keyboard-only users, though make sure that the implementation follows W3 accessibility guidelines!
@amio
Copy link
Owner

amio commented Jun 1, 2020

@jkjustjoshing Thanks for pointing out! I'm planning a rewrite, it's a good opportunity to have accessibility support baked in from start.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants