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

Pagination labels need to be more descriptive for the screen reader #734

Closed
aphelionz opened this issue Apr 29, 2018 · 0 comments · Fixed by #821
Closed

Pagination labels need to be more descriptive for the screen reader #734

aphelionz opened this issue Apr 29, 2018 · 0 comments · Fixed by #821
Labels
accessibility - WCAG AA Level AA WCAG Accessibility Criteria accessibility

Comments

@aphelionz
Copy link
Contributor

Steps to reproduce (assumes ChromeVox or similar))

  1. Open https://elastic.github.io/eui/#/navigation/pagination in a new tab.
  2. Tab to the first pagination set, number "1"

Actual Result

"One"

Expected Result

"Go to page 1" or at least "Page 1"

Category: #733: Elastic UI Context Menu Accessibility
Relevant WCAG Criteria: 2.4.6 Headings and Labels
@aphelionz aphelionz added accessibility accessibility - WCAG AA Level AA WCAG Accessibility Criteria labels Apr 29, 2018
@aphelionz aphelionz mentioned this issue Apr 29, 2018
Closed
2 tasks
snide added a commit to snide/eui that referenced this issue May 11, 2018
@snide
fixes elastic#734, adds more descriptive aria labeling for pagination
48b16d8
@snide snide mentioned this issue May 11, 2018
Merged
snide added a commit that referenced this issue May 14, 2018
@snide
Fixes for accessibility tickets (#821)
a153c7a 
* pagination labeling fixes #735
* fixes #621 adds menubar roles to side nav
* fixes #734, adds more descriptive aria labeling for pagination
* fixes #729, apply aria-live to popovers
* fixes #723, provides screen reader notice for bottom bar appearance
* fix ownFocus needing to be true for non-context menu sitations. apply aria labels depending upon ownFocus state
* fixes #616 adds home page label
* fixes #687, documentation button labeling
* fixes #617 applies aria label to theme selector
* fixes #746 applies proper label to select all checkboxes in table
* clone element and id no longer needed for popover
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
accessibility - WCAG AA Level AA WCAG Accessibility Criteria accessibility
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes for accessibility tickets snide/eui
1 participant
@aphelionz