-
Notifications
You must be signed in to change notification settings - Fork 15k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add support for keyboard initialized menu popup #38903
Conversation
Co-authored-by: Erick Zhao <erick@hotmail.ca>
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.
I'm good with this, but we should probably wait for upstream's review on the patch before merging it.
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.
API LGTM
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.
API LGTM
Co-authored-by: Samuel Maddock <samuel.maddock@gmail.com>
The code change was +1'ed upstream so landing here https://chromium-review.googlesource.com/c/chromium/src/+/4656821 |
Release Notes Persisted
|
I have automatically backported this PR to "26-x-y", please check out #38954 |
* feat: add support for keyboard initialized menu popup * Update docs/api/menu.md Co-authored-by: Erick Zhao <erick@hotmail.ca> * fix: add patch to chromium for keyboard accessibility menu behavior * refactor: s/initiatedByKeyboard/sourceType * fix: ignore initial mouse event to retain keyboard initiated focus * Update docs/api/menu.md Co-authored-by: Samuel Maddock <samuel.maddock@gmail.com> --------- Co-authored-by: Erick Zhao <erick@hotmail.ca> Co-authored-by: Samuel Maddock <samuel.maddock@gmail.com>
Solves some accessibility issues with
menu.popup
for keyboard nav accessibility usersThis PR also floats a patch that I intend to upstream. Chrome does not follow standard accessibility behaviour currently (when compared to native menus and even the //views implementation of menus that Edge uses) so this patch should go upstream to benefit all screen reader users too 馃憤
Notes: Added support for flagging a Menu as being keyboard initiated