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

Accessibility issues with Pagination (recommendations) #235

Open
roelvangils opened this issue Nov 22, 2020 · 0 comments
Open

Accessibility issues with Pagination (recommendations) #235

roelvangils opened this issue Nov 22, 2020 · 0 comments
Labels
accessibility Accessibility can be improved

Comments

@roelvangils
Copy link

Looks good overall. These recommendations would make it even better:

  • A disabled link (next and previous page) receives a different mouse-pointer and a href of null. This might cause confusion for AT users as the link can still be focused but is pronounced incorrectly by a screen reader and has an invalid hrefvalue. The easiest way to fix this is to remove the link completely.
  • The list does not state what the pagination works on. What information changes when navigating? If it's not the entire page, it should probably not be in a <nav> and the accessible name should make clear what it changes.
  • To help AT users, the number of items in the list could be mentioned. The number of items could be put in with aria-describedby and referencing the "item per page" text.
@MichaelCastiau MichaelCastiau added the accessibility Accessibility can be improved label Jan 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility Accessibility can be improved
Projects
None yet
Development

No branches or pull requests

2 participants