-
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
Add support for passive event listeners #3604
Comments
Hi! Is there any info of if/when passive listeners will land on Polymer? |
👍 |
Related: #3787 |
how to do it for polymer elements |
@azakus any progress? |
I believe this was incorrectly closed by #4829 which toggles passive gestures app-wide, whereas there needs to be a Polymer way to add passive event listeners on an ad-hoc basis, which is what this issue is about. |
[Ping] This issue should not be closed. The app-wide toggle for passive gestures has nothing to do with this. |
@azakus: can you comment on this? |
@matthew-dean Ah, that is not how I interpreted this issue, but we have no plans to support using passive gestures via syntax. If you need that, I would suggest removing |
On second thought, you were asking about adding passive gesture events I see. |
Would be really great to have this on component level in Polymer 1.x. I just checked in my app which listeners cause this message. It seems that I've this issue only with It would work without too much changes. On the other hand, what if one wants to get more information to the event creation? Registering the gesture |
@azakus Yeah, I'm working with Polymer 2 now, and you definitely can handle passive events that way. It's just not in the declarative portion of Polymer. Could be something like |
Any progress here? |
#4829 landed optional support for passive listeners on application level using |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This issue has been automatically closed after being marked stale. If you're still facing this problem with the above solution, please comment and we'll reopen! |
Echoing @RByers on pushing for support of passive event listeners which could see fairly significant scrolling performance improvements across the board.
Request on jquery: jquery/jquery#2871
Explainer: https://github.com/WICG/EventListenerOptions/blob/gh-pages/explainer.md
Chrome status page: https://www.chromestatus.com/features/5745543795965952
This hits Chrome Stable (51) in about a month.
The text was updated successfully, but these errors were encountered: