8000 Bug - Table - accessibility rules have changed, all thead cells should be `<th>`s · Issue #6272 · patternfly/patternfly · GitHub
[go: up one dir, main page]

Skip to content

Bug - Table - accessibility rules have changed, all thead cells should be <th>s #6272

@mattnolting

Description

@mattnolting

Describe the problem
Accessibility rules have changed, <td>s in theads are no longer acceptable.

How do you reproduce the problem?
Test with lighthouse, other accessibility tools, and/or screen reader

Expected behavior
Columns are described clearly and concisely with text and without altering table layout

Is this issue blocking you?

  • Replace all thead > tds (blank visually) to <th><span class="pf-v5-screen-reader">{descriptive text}{/span></th>
  • For non-text content (select all checkbox, expand all buttons), <th aria-label="Row check">{interactive element such as input or button with its own aria-label}</th>
  • Update :empty table cell styling to be something like
:is(.pf-v5-c-table__th, .pf-v5-c-table__td) {
  &:has(.pf-v5-screen-reader),
  &:empty {
    width: auto
    min-width: auto;
    padding: 0;
  }
}

Metadata

Metadata

Assignees

Type

No type

Projects

Status

Done

Relationships

None yet

Development

No branches or pull requests

Issue actions

    0