Skip to content

Improve accessibility of sidebar menu#114

Merged
jguo144 merged 2 commits intomasterfrom
jguo144/2026-03-11/update-boston-theme
Mar 12, 2026
Merged

Improve accessibility of sidebar menu#114
jguo144 merged 2 commits intomasterfrom
jguo144/2026-03-11/update-boston-theme

Conversation

@jguo144
Copy link
Collaborator

@jguo144 jguo144 commented Mar 12, 2026

Description

Improve accessibility of sidebar menu

  • Focus management: When the sidebar is opened (Enter on the menu label or click), focus moves to the first menu link after the open animation.
  • Tab order when closed: When the sidebar is closed, its links and buttons are removed from the tab order (tabindex="-1") and the panel is hidden from the accessibility tree (aria-hidden="true") so keyboard users don’t tab into off-screen content.
  • Keyboard: Enter toggles the sidebar when the menu label is focused; Escape closes it and returns focus to the menu trigger when focus was inside the sidebar.
  • A11y: Resolves the Axe “scrollable region must be keyboard accessible” issue by applying aria-hidden to .main-navigation when closed.

Copy link

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

@jguo144 jguo144 enabled auto-merge (squash) March 12, 2026 04:07
@jguo144 jguo144 merged commit e893272 into master Mar 12, 2026
5 checks passed
@jguo144 jguo144 deleted the jguo144/2026-03-11/update-boston-theme branch March 12, 2026 15:00
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.

3 participants