Thanks to visit codestin.com
Credit goes to github.com

Skip to content

perf(cdk/a11y): Don't trigger re-layout when detecting fake mousedown#28029

Merged
mmalerba merged 1 commit intoangular:mainfrom
mmalerba:fake-event
Nov 5, 2023
Merged

perf(cdk/a11y): Don't trigger re-layout when detecting fake mousedown#28029
mmalerba merged 1 commit intoangular:mainfrom
mmalerba:fake-event

Conversation

@mmalerba
Copy link
Contributor

@mmalerba mmalerba commented Oct 31, 2023

Fixes #28010

I found that testing screenX / screenY as suggested in the issue does not work. Testing detail seems to work on Chrome and Safari, but not Firefox (Firefox already had an alternate check). I wasn't able to test on Edge, but as part of the webkit family, it probably works like Chrome/Safari

Edit: @zarend helped me test some more configurations. Works with Chrome, Firefox, Safari, Edge & VoiceOver, NVDA, JAWS

@mmalerba mmalerba added perf This issue is related to performance target: patch This PR is targeted for the next patch release labels Oct 31, 2023
@mmalerba mmalerba requested a review from zarend October 31, 2023 21:03
@devversion devversion removed their request for review November 2, 2023 10:45
@mmalerba mmalerba added the action: merge The PR is ready for merge by the caretaker label Nov 5, 2023
@mmalerba mmalerba merged commit 737c7a8 into angular:main Nov 5, 2023
mmalerba added a commit that referenced this pull request Nov 5, 2023
mmalerba added a commit that referenced this pull request Nov 5, 2023
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Dec 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker perf This issue is related to performance target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(cdk/a11y): isFakeMousedownFromScreenReader - Layout thrashing (forced layout reflow)

2 participants