-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
fix(OverflowMenu): avoid trapping focus on trigger button #4688
fix(OverflowMenu): avoid trapping focus on trigger button #4688
Conversation
8b11d01
to
6d53455
Compare
Deploy preview for the-carbon-components ready! Built with commit 615012a https://deploy-preview-4688--the-carbon-components.netlify.com |
Deploy preview for carbon-elements ready! Built with commit 615012a |
Deploy preview for carbon-components-react ready! Built with commit 615012a https://deploy-preview-4688--carbon-components-react.netlify.com |
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.
Seems great, do we need to add a test for this?
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.
LGTM 👍 - Thanks @emyarod!
Closes #4659
This PR prevents the overflow menu trigger button from trapping focus
Changelog
Changed
Testing / Reviewing
Navigate to the overflow menu trigger button with the keyboard. You should be able to navigate away with Tab now