Skip to content

Improve mobile navigation accessibility#10108

Open
Plattnericus wants to merge 1 commit into
nilbuild:masterfrom
Plattnericus:fix-mobile-navigation-accessibility
Open

Improve mobile navigation accessibility#10108
Plattnericus wants to merge 1 commit into
nilbuild:masterfrom
Plattnericus:fix-mobile-navigation-accessibility

Conversation

@Plattnericus

Copy link
Copy Markdown

Summary

  • Adds expanded/hidden state attributes to the mobile navigation trigger and overlay.
  • Treats the mobile menu as a labelled modal navigation dialog.
  • Locks body scrolling while the menu is open, moves focus to the close button, restores focus to the trigger when closed, and supports closing with Escape.
  • Adds Playwright coverage for the mobile menu open and close behavior.

Validation

  • Ran ./node_modules/.bin/prettier.cmd --check src/components/Navigation/Navigation.astro src/components/Navigation/navigation.ts tests/navigation.spec.ts.
  • Ran git diff --check.
  • Transformed src/components/Navigation/navigation.ts and tests/navigation.spec.ts with esbuild to verify TypeScript syntax.

Copilot AI review requested due to automatic review settings June 29, 2026 21:09

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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.

2 participants