Skip to content

fix: Make trashcan and zoom controls accessible#9768

Merged
gonfunko merged 5 commits into
v13from
reposition
Apr 29, 2026
Merged

fix: Make trashcan and zoom controls accessible#9768
gonfunko merged 5 commits into
v13from
reposition

Conversation

@gonfunko
Copy link
Copy Markdown
Contributor

The basics

The details

Resolves

Fixes #9042

Proposed Changes

This PR makes the trashcan and zoom controls accessible via the keyboard. It does this by making them tabstops rather than part of the navigation order, since having to navigate through them with the arrow keys repeatedly would be annoying and frequently unneeded. When focused via the tab key, Enter/Space can be used to act upon them in the same way as a click. Each control has its ARIA role set to button and an appropriate label set. When the trash can is acted upon, the first block in its flyout will be focused.

@gonfunko gonfunko requested a review from a team as a code owner April 24, 2026 21:30
@gonfunko gonfunko requested a review from mikeharv April 24, 2026 21:30
@github-actions github-actions Bot added the PR: fix Fixes a bug label Apr 24, 2026
@gonfunko
Copy link
Copy Markdown
Contributor Author

This also fixes #9040

@gonfunko
Copy link
Copy Markdown
Contributor Author

This fixes #9041 as well

Copy link
Copy Markdown
Contributor

@mikeharv mikeharv left a comment

Choose a reason for hiding this comment

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

Looks like a good upgrade. I like the cleaner separation of zoom actions into discrete components. Tab-stop approach and ARIA roles/labels all make sense. It seems well aligned to the current mouse behavior.

@gonfunko gonfunko merged commit 60f423b into v13 Apr 29, 2026
4 checks passed
@gonfunko gonfunko deleted the reposition branch April 29, 2026 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: fix Fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants