Add "Main navigation" accessible name to sidebar navigation
What does this MR do and why?
Add "Main navigation" accessible name to sidebar navigation as per https://www.w3.org/WAI/ARIA/apg/practices/names-and-descriptions/
navigation
RecommendedHelps screen reader users understand the context and purpose of the navigation landmark.
Use
aria-labelledby
if a visible label is present, otherwise usearia-label
.See the Navigation Landmark section
Screenshots or screen recordings
Before | After |
---|---|
How to set up and validate locally
- Enable super sidebar (if not currently on)
- In
rails c
enable feature flagFeature.enable(:super_sidebar_nav)
- Toggle on "new navigation" in user menu
- In
- Inspect super sidebar accessibility tree
-
<nav>
should have accessible name "Main navigation"
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #394983 (closed)
Edited by Scott de Jonge