Skip to content

fix(ui): add ARIA labels to DataTable buttons - #219

Open
mdiniz97 wants to merge 1 commit into
trycompai:mainfrom
mdiniz97:fix-datatable-aria-labels
Open

mdiniz97 wants to merge 1 commit into
trycompai:mainfrom
mdiniz97:fix-datatable-aria-labels

Conversation

@mdiniz97

@mdiniz97 mdiniz97 commented Sep 14, 2026

Copy link
Copy Markdown

Added aria-label attributes to all interactive buttons in the DataTable component to improve screen reader navigation and accessibility.

Changes:

  • Mobile filters toggle button: aria-label="Toggle filters"
  • Tab selection dropdown: aria-label="Select tab: {activeTabLabel}"
  • Desktop filters button: aria-label="Filter table"
  • Sort button: aria-label="Sort table"
  • Columns toggle button: aria-label="Toggle columns"
  • Clear selection button: aria-label="Clear selection"

Benefits:

  • Improved screen reader navigation
  • Better accessibility compliance
  • Consistent ARIA labeling across all interactive elements

Fixes #218


Summary by cubic

Adds ARIA labels to all interactive buttons in the DataTable component so screen readers can announce their purpose. No visual or behavioral change.

Written for commit 7f0a0a9. Summary will update on new commits.

Review in cubic

@vercel

vercel Bot commented Sep 14, 2026

Copy link
Copy Markdown

@mdiniz97 is attempting to deploy a commit to the Comp AI - PoC Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
github-actions Bot changed the base branch from release to main September 14, 2026 04:14
@github-actions

Copy link
Copy Markdown
Contributor

Retargeted this onto main.

release is the default branch so that a plain clone runs the last tagged release, but nothing merges into it — it is fast-forwarded onto the tag by the Release workflow and that is all. Changes go to main, and reach release when a release is cut.

Nothing is wrong with your branch. If the diff now shows commits that are already on main, rebase and force-push:

git fetch origin main
git rebase origin/main
git push --force-with-lease

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

Re-trigger cubic

Added aria-label attributes to all interactive buttons in the DataTable component to improve screen reader navigation and accessibility:
- Mobile filters toggle button
- Tab selection dropdown
- Desktop filters button
- Sort button
- Columns toggle button
- Clear selection button

Fixes trycompai#218
@mdiniz97
mdiniz97 force-pushed the fix-datatable-aria-labels branch from 0079d61 to 7f0a0a9 Compare September 14, 2026 04:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve accessibility in DataTable component

1 participant