-
Notifications
You must be signed in to change notification settings - Fork 553
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
Add aria-expanded
back to AnchoredOverlay
#4456
Conversation
🦋 Changeset detectedLatest commit: 4b4e4ff The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
size-limit report 📦
|
@TylerJDev Hi! Do you have an idea of when this will be merged? 🙏🏻 |
@lindseywild, just needs to get reviewed now! I redid my integration test PR, and it looks good (outside of flakey tests). Requires some changes in GH for tests: https://github.com/github/github/pull/323711#issuecomment-2098990100. cc: @primer/engineer-reviewers for a review 👀 |
@primer/engineer-reviewers Can someone please review this when you get a chance? Thanks! |
Will resolve: https://github.com/github/accessibility-audits/issues/6932
We have an open accessibility issue related to how we handle
aria-expanded
withinAnchoredOverlay
. We did not applyaria-expanded
when a menu was open since it was redundant, and it was very unlikely that there'd be navigation outside of the menu before it was closed. We've received feedback that this may cause an unintended effect for users using specific devices/AT that makes it difficult to leave the menu.Changelog
Changed
aria-expanded
state toAnchoredOverlay
.Rollout strategy
Integration test PR: https://github.com/github/github/pull/323711. There are some flakey tests that are failing which are not caused by this PR. The changes required for this PR are: https://github.com/github/github/pull/323711/commits/dd6a1dee058ee4659bb1ed36ac50e5356b3de832
Testing & Reviewing
Merge checklist