Skip to content

refactor(aria/toolbar): remove selection behavior - #33747

Merged
ok7sai merged 1 commit into
angular:mainfrom
ok7sai:refactor/toolbar-remove-selection
Sep 2, 2026
Merged

refactor(aria/toolbar): remove selection behavior#33747
ok7sai merged 1 commit into
angular:mainfrom
ok7sai:refactor/toolbar-remove-selection

Conversation

@ok7sai

@ok7sai ok7sai commented Aug 28, 2026

Copy link
Copy Markdown
Member

Removing selection behavior so Toolbar no longer listen to Enter and Space events.

Fixes #33684

@ok7sai
ok7sai force-pushed the refactor/toolbar-remove-selection branch from 266db1f to 3ddd157 Compare August 28, 2026 23:31
@ok7sai
ok7sai marked this pull request as ready for review September 1, 2026 17:51

@crisbeto crisbeto left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Do we need to update ADEV as well?

@ok7sai

ok7sai commented Sep 1, 2026

Copy link
Copy Markdown
Member Author

LGTM. Do we need to update ADEV as well?

Yes we have to. I can update ADEV examples first to decouple the selection to be safe.

@ok7sai ok7sai added action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release labels Sep 1, 2026
@ok7sai
ok7sai merged commit 0b81ad7 into angular:main Sep 2, 2026
36 checks passed
@ok7sai

ok7sai commented Sep 2, 2026

Copy link
Copy Markdown
Member Author

This PR was merged into the repository. The changes were merged into the following branches:

@ok7sai
ok7sai deleted the refactor/toolbar-remove-selection branch September 2, 2026 06:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: aria/toolbar target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(aria/toolbar): Support Select/Comboboxes in the toolbar

2 participants