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

Combo Box is a keyboard trap #788

Closed
aphelionz opened this issue May 6, 2018 · 2 comments · Fixed by #866
Closed

Combo Box is a keyboard trap #788

aphelionz opened this issue May 6, 2018 · 2 comments · Fixed by #866
Assignees
@nreese
Labels
accessibility - WCAG A Level A WCAG Accessibility Criteria accessibility

Comments

@aphelionz
Copy link
Contributor

Steps to reproduce (assumes ChromeVox or similar)

image

  1. Open https://elastic.github.io/eui/#/forms/combo-box in a new tab.
  2. Tab to the any demo combo box, and try to tab through the button highlighted in the picture

Actual Result

Keyboard is trapped there, user is unable to tab forward from that point

Expected Result

User is able to tab through the button

Really good accessibility work done on this element though!!
Category: #786: Elastic UI Combo Box Accessibility
Relevant WCAG Criteria: 2.1.2 No Keyboard Trap
@aphelionz aphelionz added accessibility accessibility - WCAG A Level A WCAG Accessibility Criteria labels May 6, 2018
@aphelionz aphelionz mentioned this issue May 6, 2018
Closed
3 tasks
@timroes
Copy link
Contributor

timroes commented May 7, 2018

Also it seems, that Tab doesn't do anything while focused on the input box?

@nreese nreese self-assigned this May 17, 2018
@nreese nreese mentioned this issue May 17, 2018
Closed
@cjcenizal
Copy link
Contributor

cjcenizal commented May 22, 2018 •
edited
Loading

Really good accessibility work done on this element though!!

WOOHOO! Thanks Mark!

@cjcenizal cjcenizal mentioned this issue May 23, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nreese nreese

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.

Fix EuiComboBox focus trap cjcenizal/eui
Combo Box is a keyboard trap nreese/eui
4 participants
@aphelionz @nreese @timroes @cjcenizal