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

[EuiContextMenu] Items should be nested within an ARIA-compatible menu structure #731

Closed
aphelionz opened this issue Apr 29, 2018 · 4 comments
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/context-menu in a new tab.
  2. Tab to the "Click me to load a context menu" button
  3. Press enter and listen to the screen reader

Actual Result

"View Options Show Fullscreen Share this Dashboard Edit / add Panels Display Options"

Expected Result

"View Options, Show Fullscreen, Button Selected 1 of 4"

Category: #730: Elastic UI Context Menu Accessibility
Relevant WCAG Criteria: 3.2.3 Consistent Navigation

@aphelionz aphelionz added accessibility accessibility - WCAG AA Level AA WCAG Accessibility Criteria labels Apr 29, 2018
@cchaos cchaos changed the title Context Menu Items should be nested within an ARIA-compatible menu structure [EuiContextMenu] Items should be nested within an ARIA-compatible menu structure Mar 18, 2020
@snide snide mentioned this issue Mar 9, 2021
41 tasks
@cchaos
Copy link
Contributor

cchaos commented Mar 10, 2021

@myasonik Are there any guidelines on how to implement this properly?

@aphelionz
Copy link
Contributor Author

Seems a lot has changed + been debated since then but I'd look at this article: https://adrianroselli.com/2019/06/link-disclosure-widget-navigation.html

@github-actions
Copy link

github-actions bot commented Sep 7, 2021

👋 Hey there. This issue hasn't had any activity for 180 days. We'll automatically close it if that trend continues for another week. If you feel this issue is still valid and needs attention please let us know with a comment.

@cee-chen
Copy link
Member

cee-chen commented Apr 8, 2022

Closing this in favor of #5737, since WAI ARIA examples/specs have changed since this issue was opened.

@cee-chen cee-chen closed this as completed Apr 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility - WCAG AA Level AA WCAG Accessibility Criteria accessibility
Projects
None yet
Development

No branches or pull requests

3 participants