-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Perf: Do not add document-level blocking touch listeners #3787
Comments
Back in the day I know @azakus worked to ensure this wasn't necessary (it was the main original concern from abarth@ about the PointerEventPolyfill IIRC). We're pretty close to forcing all touch listeners on the document to be passive by default anyway - you can enabled this in chrome://flags, and I haven't noticed any issues with polymer. So perhaps it's easy to just make these listeners passive? |
What can we do about this? Any idea? |
What is going here and why is there no final answer? I'm getting thousands of console warnings: |
Is this the |
Just wondering if there's any updates on this?! |
All polymer apps get a touch listener added to
document
by default?Doing so is typically frowned upon b/c it can have negative effects on page load and scrolling perf, especially on mobile. What are these events for? Can they be passive events by default...or can we have users opt-into passive events somehow if they know they're not going to call
preventDefault()
?https://github.com/WICG/EventListenerOptions/blob/gh-pages/explainer.md
cc @RByers @azakus
The text was updated successfully, but these errors were encountered: