Accessibility: Mobile nav trigger missing aria-expanded #192

New Issue

Closed

opened 2026-05-29 16:45:54 -05:00 by null · 1 comment

null commented

2026-05-29 16:45:54 -05:00

Owner

Severity: Minor

Description: The SheetTrigger button in MobileNav lacks aria-expanded to indicate menu state. Screen readers cannot communicate whether the menu is open or closed.

File: src/components/layout/MobileNav.jsx line 26

Fix direction: Add aria-expanded={isOpen} and aria-controls="mobile-nav-content" to the trigger button.

**Severity:** Minor **Description:** The SheetTrigger button in MobileNav lacks `aria-expanded` to indicate menu state. Screen readers cannot communicate whether the menu is open or closed. **File:** `src/components/layout/MobileNav.jsx` line 26 **Fix direction:** Add `aria-expanded={isOpen}` and `aria-controls="mobile-nav-content"` to the trigger button.

null added the

bug

accessibility

labels 2026-05-29 16:45:54 -05:00

null added this to the Batch 10 — Accessibility WCAG 2.1 AA milestone 2026-08-18 01:24:53 -05:00

null referenced this issue from a commit

2026-08-18 02:31:20 -05:00

fix(a11y): visible focus indicator, aria-expanded, and six Label-in-Name failures

null commented

2026-08-18 02:31:33 -05:00

Author

Owner

Fixed in 9fdc6b6. src/components/layout/MobileNav.jsx: the SheetTrigger button now carries aria-expanded={isOpen} and aria-controls='mobile-nav-content', and SheetContent carries the matching id.

Verified: dist/index.html contains aria-expanded="false" and id="mobile-nav-content" in the prerendered markup, so it is correct before hydration as well as after.

Fixed in 9fdc6b6. src/components/layout/MobileNav.jsx: the SheetTrigger button now carries aria-expanded={isOpen} and aria-controls='mobile-nav-content', and SheetContent carries the matching id. Verified: dist/index.html contains aria-expanded="false" and id="mobile-nav-content" in the prerendered markup, so it is correct before hydration as well as after.

null closed this issue

2026-08-18 02:31:33 -05:00