fix(multiple): change aria keyboard manager to only handle repeated events in correct places #32728
+79
−44
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
adolgachev
changed the title
adolgachev
force-pushed
the
aria-kb-repeat
branch
from
366dfa3 to
4c5c3e0
Compare
adolgachev
changed the title
adolgachev
force-pushed
the
aria-kb-repeat
branch
from
4c5c3e0 to
f75d2e2
Compare
adolgachev
changed the title
adolgachev
added
Accessibility
adolgachev
force-pushed
the
aria-kb-repeat
branch
from
e65f6c8 to
35c6305
Compare
tjshiu
reviewed
Feb 9, 2026
| * This library has not yet had a need for stopPropagationImmediate. | ||
| */ | ||
| export interface EventHandlerOptions { | ||
| handleRepeat?: boolean; |
Contributor
There was a problem hiding this comment.
I might prefer preventRepeat instead. It makes it clearer what it is doing. I wasn't sure at first what handleRepeat was doing.
Contributor
Author
There was a problem hiding this comment.
I originally had ignoreRepeat but then I changed the default to ignore the repeat, and it is generally better to have defaults be false.
But we actually do default the others as true already, so I'll go back to that.
adolgachev
force-pushed
the
aria-kb-repeat
branch
from
2df86f6 to
ee8f248
Compare
tjshiu
approved these changes
Feb 10, 2026
…vents in correct places
adolgachev
force-pushed
the
aria-kb-repeat
branch
from
d898b53 to
c617fd7
Compare
adolgachev
added
action: merge
Contributor
Author
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The angular-aria keyboard manager ignores the event.repeat parameter so holding down a key will cause it to loop even when that behavior is not desired. In general, repeated events should only be handled when navigating with arrow keys, and especially not when toggling (as then it just goes from off to on to off and so on).
Another issue is that the browser can interpret a keyboard space or enter as a click in certain cases. This is true with the toolbar so added check for a proper mouse event to handle the click (else already handled with keyboard) as well as updated spec to pass a proper event.
Note: we are missing tests for this but they probably need to be added at a higher level so that we can catch for not just the repeat issue but also issues like the click above. Else can just add explicit repeat but keyDown events but that isn't as useful.
Fixes b/479281429.