-
Notifications
You must be signed in to change notification settings - Fork 977
Support passive event handlers for mouse/touch events #162
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
I am using
for zoom which uses the mouse wheel listener, it has to be active listener so default action (which is page scroll) can be prevented. For the rest of the events I guess we can benefit from this once React and d3 starts to support it. |
I'm getting this error in chrome when I scroll to zoom.
I expect it to only zoom on the chart but it zooms and scrolls the page at the same time. |
Same here. |
Same error in Chrome. I'm using nextjs. |
Just to keep pace with the react team, maybe adding these options to the scroll/wheel listeners could boost performance..
facebook/react#6436
The text was updated successfully, but these errors were encountered: