-
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
Adding passive event listeners #4393
Comments
I analyzed my app with Lighthouse it suggest to uses passive listeners to improve scrolling performance. |
@khammami I got the same warning too. But the one adding non-passive listeners is polymer itself.. and some iron component. How should we replace that? |
Hi @wingliu0, here what I found about passive events. touch listeners should be annotated with {passive: true}. I believe the best way to replace them is by reporting each non passive listener you find to the team. |
@khammami true, but will they still maintain the polymer 1.x? |
@wingliu0 I think so, at least till the final release of v2 |
Dupe of #3604 |
Description
DevTool of chrome canary in
verbose
mode gives a violationThis violation comes for
touchstart
andmousewheel
eventsSteps to Reproduce
chrome canary
console
ofdevTool
switch level to verbose.Versions
The text was updated successfully, but these errors were encountered: