Skip to content

fix(settings): add aria-label to icon-only edit/delete buttons on AI provider keys#4922

Merged
pedrofrxncx merged 1 commit into
mainfrom
fix/ai-provider-key-icon-buttons-aria-label-w4
Jul 21, 2026
Merged

fix(settings): add aria-label to icon-only edit/delete buttons on AI provider keys#4922
pedrofrxncx merged 1 commit into
mainfrom
fix/ai-provider-key-icon-buttons-aria-label-w4

Conversation

@pedrofrxncx

@pedrofrxncx pedrofrxncx commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

Why

This follows the same accessibility hardening pattern as #4904, #4907, and #4915 (which added aria-labels to icon-only actions-menu, KPI-chart, and mic/send buttons). The Edit and Delete buttons on each AI provider key row (apps/mesh/src/web/views/settings/ai-providers/provider-key-row.tsx) render only an Edit01/Trash01 icon with no aria-label, no title, and no wrapping Tooltip — so a screen reader announces both as an unlabeled "button" in a list where every row's controls are visually identical, giving no way to distinguish edit from delete non-visually.

Fix

Added aria-label="Edit provider key" and aria-label="Delete provider key" to the two icon-only Buttons. Pure additive attribute change — no logic, styling, or behavior touched.

Verification

  • bun run fmt — clean
  • cd apps/mesh && bunx tsc --noEmit — clean (exit 0)
  • No existing test covers this component's rendering; the change is a static JSX attribute addition with no logic branch, so no regression test is needed. Full CI validates the rest.

How to confirm

Open Org Settings → AI Providers, add a provider key, then inspect the Edit/Delete icon buttons in the accessibility tree (or with a screen reader) — each now announces its action instead of just "button".


Summary by cubic

Add aria-labels to the icon-only Edit and Delete buttons on AI provider key rows so screen readers announce the correct action. They now read "Edit provider key" and "Delete provider key" instead of an unlabeled "button".

Written for commit 17408fd. Summary will update on new commits.

Review in cubic

…provider keys

The edit and delete buttons on each AI provider key row render only an icon
(Edit01 / Trash01) with no accessible name, so a screen reader announces them
as unlabeled "button" controls in a list where every row looks identical.
@pedrofrxncx
pedrofrxncx enabled auto-merge (squash) July 21, 2026 13:14
@pedrofrxncx
pedrofrxncx merged commit fbd3bac into main Jul 21, 2026
15 of 16 checks passed
@pedrofrxncx
pedrofrxncx deleted the fix/ai-provider-key-icon-buttons-aria-label-w4 branch July 21, 2026 13:16
decocms Bot pushed a commit that referenced this pull request Jul 21, 2026
PR: #4922 fix(settings): add aria-label to icon-only edit/delete buttons on AI provider keys
Bump type: patch

- decocms (apps/mesh/package.json): 4.82.15 -> 4.82.16

Deploy-Scope: web
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.

1 participant