Menu List component could use a11y improvements
Open
Nobody has claimed this yet.
A11y ♿️
Bug :beetle:
- Dominant language
- SCSS
- Stars
- 298
- Forks
- 83
- Avg merge
- 16h 48m
- Merged PRs (30d)
- 1
Description
Description
The menu list component could benefit from some a11y improvements.
Steps to reproduce
- Visit https://protocol.mozilla.org/patterns/molecules/menu-list.html
- Using the keyboard,
tabso the menu list is focused. - Hit
enterto open the menu.
Expected result
- If i hit
escwhen open, it should close the menu. - If I
clickanywhere outside of the menu whilst open, the menu should close again. - If I
tabthrough the menu items and past the last item in the list, then the menu should close when focus leaves the element.
Actual result
The menu remains open for all of the above items. The only way to close a menu currently is to click the title again.
Environment
Firefox Nightly (but happens in any browser)
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the menu list component shown on the linked Menu List page and reproduce the keyboard and focus behavior in the issue. Check how the menu responds to Escape, outside clicks, and focus moving past the last item. Done means each action closes the open menu consistently across browsers.
Written by the indexing model from the issue text.
Assessment
- Domain
- accessibility, frontend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100