Library
React Components / v9 (@fluentui/react-components)
System Info
Are you reporting Accessibility issue?
None
Reproduction
N/A
Bug Description
Actual Behavior
Listeners are not passive:
|
targetWindow.addEventListener('scroll', updatePosition); |
|
targetWindow.addEventListener('resize', updatePosition); |
Expected Behavior
Listeners should be passive for better performance:
https://developer.chrome.com/docs/lighthouse/best-practices/uses-passive-event-listeners/
Logs
Requested priority
High
Products/sites affected
N/A
Are you willing to submit a PR to fix?
yes
Validations
Library
React Components / v9 (@fluentui/react-components)
System Info
Are you reporting Accessibility issue?
None
Reproduction
N/A
Bug Description
Actual Behavior
Listeners are not passive:
fluentui/packages/react-components/react-positioning/src/createPositionManager.ts
Lines 133 to 134 in d1a32c7
Expected Behavior
Listeners should be passive for better performance:
https://developer.chrome.com/docs/lighthouse/best-practices/uses-passive-event-listeners/
Logs
Requested priority
High
Products/sites affected
N/A
Are you willing to submit a PR to fix?
yes
Validations