Skip to content

refactor(frontend): accessible names for the icon-only buttons - #4011

Merged
Kadrian merged 1 commit into
refactor/toggle-button-groupfrom
refactor/accessible-names
Sep 10, 2026
Merged

refactor(frontend): accessible names for the icon-only buttons#4011
Kadrian merged 1 commit into
refactor/toggle-button-groupfrom
refactor/accessible-names

Conversation

@Kadrian

@Kadrian Kadrian commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

Design-system prep, step 10: accessible names for the icon-only buttons

Stacked on #4009. Step 7 gave the icon-only buttons inside a tooltip their tooltip text as aria-label; the ones without a tooltip get theirs here, so every button in the app has a name for screen readers and tests.

  • 13 buttons named: retry a concept tree, page through the preview charts, previous and next month in the datepicker, the help menu, edit folders, remove a concept from a node, the remove and add buttons in the forms
  • new keys common.add, common.retry, preview.previousPage / nextPage, inputDate.previousMonth / nextMonth, in both languages
  • the trigger docs on Tooltip, Menu and ConfirmMenu name Button and ToggleButton

@Kadrian
Kadrian added this pull request to stack #4004 September 9, 2026 09:02
@Kadrian Kadrian changed the title refactor(frontend): every icon-only button has an accessible name; trigger docs name Button and ToggleButton refactor(frontend): accessible names for the icon-only buttons Sep 9, 2026
@Kadrian
Kadrian marked this pull request as ready for review September 9, 2026 09:05
@Kadrian
Kadrian force-pushed the refactor/accessible-names branch from 624b499 to 18dfd18 Compare September 9, 2026 09:11
@Kadrian
Kadrian force-pushed the refactor/accessible-names branch from 18dfd18 to 624b499 Compare September 9, 2026 09:38
@Kadrian

Kadrian commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator Author

@awildturtok @thoniTUB can one of you have a look here?

@Kadrian
Kadrian force-pushed the refactor/accessible-names branch from 624b499 to 7e529a8 Compare September 10, 2026 12:27
@Kadrian
Kadrian added this pull request to the merge queue Sep 10, 2026
Merged via the queue into develop with commit 21b496c Sep 10, 2026
19 checks passed
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.

2 participants