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: +328 B (+0.01%)

Total Size: 4.24 MB

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

compressed-size-action

@Ovgodd Ovgodd force-pushed the fix/docs-grid-aria-list-pattrn branch 2 times, most recently from b536786 to 18b4302 Compare March 25, 2026 15:09
Ovgodd added 3 commits March 25, 2026 18:03
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 18b4302 to 5cccc1a Compare March 25, 2026 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: In review

1 participant