Accessibility: icon-only buttons missing aria-label across asset and library screens #126

New issue

Closed

opened 2026-05-26 18:22:16 -04:00 by zgaetano · 1 comment

zgaetano commented

2026-05-26 18:22:16 -04:00

Owner

Fixed in 04ce096. Icon-only buttons across asset detail, library, projects, schedule, admin, settings, and editor screens now carry descriptive aria-label attributes (Close, More actions, Project actions, User actions, Asset actions, Play/Pause, Toggle mute, Toggle fullscreen, Show/Hide password, Sign out, Remove member, Create bin, Delete recorder, Previous/Next day, Toggle sidebar, etc.).

Fixed in 04ce096. Icon-only buttons across asset detail, library, projects, schedule, admin, settings, and editor screens now carry descriptive `aria-label` attributes (Close, More actions, Project actions, User actions, Asset actions, Play/Pause, Toggle mute, Toggle fullscreen, Show/Hide password, Sign out, Remove member, Create bin, Delete recorder, Previous/Next day, Toggle sidebar, etc.).

~~zgaetano referenced this issue 2026-05-26 18:22:49 -04:00~~

Tracker: 2026-05-26 audit session — 27 bugs logged (#100-#127) #128

zgaetano commented

2026-05-26 19:36:13 -04:00

Author

Owner

Fix Plan — #126 Icon-only buttons missing aria-label

Root cause: Icon-only <button className="icon-btn"> elements have no aria-label or title. Screen readers announce "button" with no context.

Locations: screens-asset.jsx:329,344,482, screens-library.jsx:349,617.

Fix — add aria-label to every icon-only button:

// before:
<button className="icon-btn" onClick={handleDelete}><TrashIcon /></button>

// after:
<button className="icon-btn" onClick={handleDelete} aria-label="Delete asset"><TrashIcon /></button>

Audit all <button> elements — if no text content, must have aria-label.

Files: screens-asset.jsx, screens-library.jsx
Effort: ~30min
**Priority: P3 — accessibility

## Fix Plan — #126 Icon-only buttons missing aria-label **Root cause:** Icon-only `<button className="icon-btn">` elements have no `aria-label` or `title`. Screen readers announce "button" with no context. **Locations:** `screens-asset.jsx:329,344,482`, `screens-library.jsx:349,617`. **Fix — add aria-label to every icon-only button:** ```js // before: <button className="icon-btn" onClick={handleDelete}><TrashIcon /></button> // after: <button className="icon-btn" onClick={handleDelete} aria-label="Delete asset"><TrashIcon /></button> ``` Audit all `<button>` elements — if no text content, must have `aria-label`. **Files:** `screens-asset.jsx`, `screens-library.jsx` **Effort:** ~30min **Priority: P3 — accessibility

zgaetano referenced this issue from a commit

2026-05-26 22:06:25 -04:00

chore: 1.2 ship-prep sweep — close 38 issues

zgaetano closed this issue

2026-05-26 22:07:54 -04:00

~~zgaetano referenced this issue 2026-05-28 19:31:35 -04:00~~

UI Audit: Projects screen has 2 icon-only buttons missing aria-label #154