Skip to content

♿️(frontend) replace ARIA grid pattern with list in docs grid#2131

Open
Ovgodd wants to merge 3 commits intomainfrom
fix/docs-grid-aria-list-pattrn
Open

♿️(frontend) replace ARIA grid pattern with list in docs grid#2131
Ovgodd wants to merge 3 commits intomainfrom
fix/docs-grid-aria-list-pattrn

Conversation

@Ovgodd
Copy link
Collaborator

@Ovgodd Ovgodd commented Mar 25, 2026

Purpose

The document grid used role="grid" with role="row", role="gridcell", role="rowgroup", and role="columnheader". This ARIA grid pattern requires full keyboard navigation with arrow keys which it's not implemented and demand to much complexity, . With role="grid", some screen readers switch to a mode where arrow keys are handled by the page; since the page did not implement grid navigation, that interaction did not work as expected.

Proposal

  • Replace role="grid" / role="row" / role="gridcell" / role="columnheader" / role="rowgroup" with role="list" / role="listitem", matching the actual semantics (a list of documents, not a data grid)
  • Mark visual column headers (Name, Updated at) as aria-hidden="true" since they are no longer part of the list semantics
  • Add translatable screenReaderInstructions and announcements on DndContext (accessibility prop) so SR copy can be localized via i18n

@github-actions
Copy link

github-actions bot commented Mar 25, 2026

Size Change: +337 B (+0.01%)

Total Size: 4.25 MB

Filename Size Change
apps/impress/out/_next/static/1e155e41/_buildManifest.js 621 B +621 B (new file) 🆕
apps/impress/out/_next/static/aa40bd0a/_buildManifest.js 0 B -622 B (removed) 🏆

compressed-size-action

@Ovgodd Ovgodd force-pushed the fix/docs-grid-aria-list-pattrn branch 3 times, most recently from 18b4302 to 5cccc1a Compare March 25, 2026 17:04
@Ovgodd Ovgodd force-pushed the fix/docs-grid-aria-list-pattrn branch from 5cccc1a to 8128a51 Compare March 27, 2026 09:15
Ovgodd added 3 commits March 27, 2026 10:38
Use list/listitem roles, mark column headers as aria-hidden
Pass i18n strings for screenReaderInstructions and announcements.
Adapt selectors from grid/row to list/listitem and add a Tab+Enter test
@Ovgodd Ovgodd force-pushed the fix/docs-grid-aria-list-pattrn branch from 8128a51 to 210c8b5 Compare March 27, 2026 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: In review

2 participants