Skip to content

fix(frontend): trap focus in mobile navigation drawer - #590

Merged
dark-sarge merged 1 commit into
arflexx:mainfrom
Fi-zzy:fix/issue-456
Oct 26, 2026
Merged

dark-sarge merged 1 commit into
arflexx:mainfrom
Fi-zzy:fix/issue-456

Conversation

@Fi-zzy

@Fi-zzy Fi-zzy commented Oct 25, 2026

Copy link
Copy Markdown

Summary

  • Keep the mobile navigation dialog out of the accessibility tree while closed.
  • Move focus into the drawer when opened, trap Tab and Shift+Tab while open, and return focus to the menu trigger when it closes.
  • Close the drawer on Escape and navigation; add keyboard and axe regression coverage.

Fixes #456

Validation

  • pnpm --filter airflex-frontend exec jest --runInBand components/Navbar.test.tsx (8 tests passed; axe audit passed)
  • pnpm --filter airflex-frontend lint reports existing TypeScript errors in unrelated page metadata and sell-page files.

@Fi-zzy
Fi-zzy requested a review from dark-sarge as a code owner October 25, 2026 19:34
@dark-sarge
dark-sarge merged commit a82778e into arflexx:main Oct 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Frontend] - Add Keyboard Navigation Support for Navbar Menu

2 participants