Skip to content

♿️(frontend) fix sidebar resize handle for screen readers#2122

Open
Ovgodd wants to merge 1 commit intomainfrom
fix/separator-keyboard-interact
Open

♿️(frontend) fix sidebar resize handle for screen readers#2122
Ovgodd wants to merge 1 commit intomainfrom
fix/separator-keyboard-interact

Conversation

@Ovgodd
Copy link
Collaborator

@Ovgodd Ovgodd commented Mar 24, 2026

Purpose

Fix left sidebar keyboard resize with a screen reader on, and improve SR output plus step size.

Proposal

  • Replace the default separator role with slider so that, when the handle is focused, screen readers pass Left/Right to the app instead of using them for reading (known issue with separator in NVDA).
  • from aria-valuenow into aria-valuetext (width in px).
  • Use keyboardResizeBy={1} for smaller steps.

@Ovgodd Ovgodd requested a review from AntoLC March 24, 2026 15:00
@Ovgodd Ovgodd self-assigned this Mar 24, 2026
@Ovgodd Ovgodd force-pushed the fix/separator-keyboard-interact branch from aa9cacf to c22927b Compare March 24, 2026 15:01
@Ovgodd Ovgodd marked this pull request as ready for review March 24, 2026 15:03
@Ovgodd Ovgodd moved this from Backlog to In review in LaSuite Docs A11y Mar 24, 2026
@Ovgodd Ovgodd linked an issue Mar 24, 2026 that may be closed by this pull request
Expose the handle as a slider so arrow keys work with NVDA
@Ovgodd Ovgodd force-pushed the fix/separator-keyboard-interact branch from c22927b to b7d0ba7 Compare March 24, 2026 15:55
@github-actions
Copy link

Size Change: +214 B (+0.01%)

Total Size: 4.23 MB

Filename Size Change
apps/impress/out/_next/static/3574adc1/_buildManifest.js 0 B -905 B (removed) 🏆
apps/impress/out/_next/static/f3d1265a/_buildManifest.js 904 B +904 B (new file) 🆕

compressed-size-action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

Resizable separator: inactive with keyboard

1 participant