fix(Table): drop row tree ARIA outside treegrid

[AKnassa]

Part of #4681 (weekly a11y scan). Clears all 10 aria-conditional-attr baseline entries (TableTree x7, TableGroupedRows x3), plus the identical violation in the never-baselined TableTree::Row Click Expansion story.

What this does

The tree and groupedRows Table plugins set aria-expanded (and the tree plugin aria-level) on <tr> elements. Those attributes are only valid on rows inside a treegrid, and Astryx Table is a native <table>, so axe flags every story. Rows now carry no tree ARIA; the expand/collapse state stays announced by the chevron buttons, which already carry aria-expanded (the same pattern the rowExpansion plugin uses, which has zero baseline entries).

Why not role="treegrid" instead

Treegrid contracts the full ARIA grid keyboard model (roving tabindex, arrow-key cell navigation, focusable cells), none of which BaseTable implements. Screen readers would switch users into grid interaction mode and hand them an unnavigable widget. Building that model is a feature decision, not this fix. For groupedRows it is also semantically wrong: group headers are section separators of flat data, not tree parents.

Trade-off, stated plainly

Tree depth is now conveyed visually by indentation only. The removed aria-level was invalid in this context (and unreliably announced because of it), but assistive technology currently gets no programmatic depth cue. A follow-up candidate is announcing the level through the expander's accessible name; the changeset records the same trade-off so it lands as a decision, not a side effect.

What changed

• Rows lose aria-expanded (both plugins) and aria-level (tree plugin).
• The tree plugin's per-row ref/subscription machinery existed solely to apply row ARIA and is removed with it (net -77 lines); row-click expansion is unaffected and a dedicated regression review of the removal found no other dependents.
• Tests rewritten to assert rows never carry these attributes; the chevron buttons remain the single home of the expand state.
• doc.mjs updated to describe the actual semantics, including the visual-only depth limitation.
• Baseline: 10 entries removed, deletions only. Changeset (patch) includes a tr[aria-expanded] / tr[aria-level] test-migration hint.

Observation for a follow-up

Local audits report TableGroupedRows::Default::color-contrast and TableGroupedRows::Initially Collapsed::color-contrast as no longer occurring (stale baseline entries). Left untouched here; contrast entries are out of this PR's scope.

Verification

474/474 tests across all 22 Table test files, core typecheck, eslint clean, and the repo's axe audit with --fail-on-new: 0 violations across all 12 TableTree/TableGroupedRows stories.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
astryx	Ready	Preview	Aug 3, 2026 9:43pm