-
Notifications
You must be signed in to change notification settings - Fork 13
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
feat(flyout-menu): accessibility improvements for flyout menu component #1495
base: master
Are you sure you want to change the base?
Conversation
🚀 Deployed on https://pr-1495--dhis2-ui.netlify.app |
Passing run #3355 ↗︎Details:
Review all test suite changes for PR #1495 ↗︎ |
0675f99
to
0342c0c
Compare
0342c0c
to
de7d976
Compare
de7d976
to
94f69df
Compare
d14f666
to
f31fd08
Compare
841d270
to
3a6e749
Compare
3a6e749
to
51a0207
Compare
51a0207
to
b930acb
Compare
- focus the popper's first child element when it is rendered - move focus to flyout menu's first child, i.e. the menu when it receives focus - the Right arrow key opens the submenu, Left arrow key closes submenu and focuses its parent item - close flyout menu with the Escape key by passing it a closeMenu function as a prop - add unit tests simulating opening and closing of submenus
b930acb
to
9c66c4d
Compare
Hi @d-rita |
In order to reproduce this, I have tested with one of the
with.menu.mov
options.menu.mov
modified.options.menu.movConclusion:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for investigating the root cause @d-rita. Nice digging!
I am approving the pr based on your conclusions.
Implements LIBS-563
Description
This feature improves the accessibility of the flyout menu component
Escape
keyRight Arrow
key on a focused menu item containing a submenu, a flyout menu is opened and focus is placed on the 1st item.Left Arrow
key on any focused item inside a submenu, it will close and focus the parent menu item if any.Checklist
All points above should be relevant for feature PRs. For bugfixes, some points might not be relevant. In that case, just check them anyway to signal the work is done.
Screenshots
supporting text