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

Filtered Menu Lacks Context #621

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

Filtered Menu Lacks Context #621

aphelionz opened this issue Apr 2, 2018 · 0 comments · Fixed by #821
Assignees
@snide
Labels
accessibility - WCAG A Level A WCAG Accessibility Criteria accessibility

Comments

@aphelionz
Copy link
Contributor

aphelionz commented Apr 2, 2018 •
edited
Loading

#Steps to reproduce (assumes ChromeVox or similar)

image

  1. [Tab] or [Shift+Tab] to the search field
  2. Enter the letters "on" to filter the list
  3. Press [Tab] to navigate to a navigation section with more than one result, such as the one shown above
  4. Listen to the screen reader

Actual Result

"Button, button"

That is, the text of the link and the type.

Expected Result

Button, Navigation List, 1 of 3, button

Suggested Fixes
Convert the markup of these lists to proper navigation either by changing the HTML tags or by adding aria-role="navigation"

Note that this problem persists in the submenus too:
image

Category: #615: Elastic UI Chrome Accessibility
Relevant WCAG Criteria: 3.3.2 Labels or Instructions and 3.2.3 Consistent Navigation
@aphelionz aphelionz mentioned this issue Apr 2, 2018
Closed
9 tasks
@aphelionz aphelionz mentioned this issue Apr 29, 2018
Closed
1 task
@snide snide self-assigned this May 4, 2018
@aphelionz aphelionz added the accessibility - WCAG A Level A WCAG Accessibility Criteria label May 6, 2018
snide added a commit to snide/eui that referenced this issue May 11, 2018
@snide
fixes elastic#621 adds menubar roles to side nav
e39a031
@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

@snide snide

Labels
accessibility - WCAG A Level A 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
2 participants
@aphelionz @snide