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

Dialogs Need to Signal Context Switch to Users Upon Opening #712

Closed
aphelionz opened this issue Apr 28, 2018 · 1 comment
Closed

Dialogs Need to Signal Context Switch to Users Upon Opening #712

aphelionz opened this issue Apr 28, 2018 · 1 comment
Labels
accessibility - WCAG A Level A WCAG Accessibility Criteria accessibility documentation Issues or PRs that only affect documentation - will not need changelog entries

Comments

@aphelionz
Copy link
Contributor

Steps to reproduce (assumes ChromeVox or similar)

  1. Open https://elastic.github.io/eui/#/guidelines/modals in a new tab.
  2. Tab through the navigation to the main content pane.
  3. Tab to the "Show Component Code" button and press enter
  4. Tab to the "Show Confirm Modal" button
  5. Press enter

Actual Result

Only the "Yes, Do It" button is focused and read

Expected Result

The screen reader should read information about the dialog itself to and provide some instructions on how to leave the dialog if they want i.e. "Do this thing Dialog. Press esc to exit"

Suggested Fix
Try the example here: http://gdkraus.github.io/accessible-modal-dialog/ with ChromeVox

Category: #699: Elastic UI Modals Accessibility
Relevant WCAG Criteria: 3.2.2 Predictable on Input

@aphelionz aphelionz added accessibility accessibility - WCAG A Level A WCAG Accessibility Criteria labels Apr 28, 2018
@aphelionz aphelionz mentioned this issue Apr 28, 2018
2 tasks
@snide snide added the documentation Issues or PRs that only affect documentation - will not need changelog entries label Jun 8, 2018
@chandlerprall
Copy link
Contributor

Modal guideline page no longer exists. Modal accessibility - including keyboard and screen reader support - is now being tracked in #3889

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility - WCAG A Level A WCAG Accessibility Criteria accessibility documentation Issues or PRs that only affect documentation - will not need changelog entries
Projects
None yet
Development

No branches or pull requests

3 participants