Add warning message in DevTools when a touch event listener is not passive |
||||||
Issue description
It would be nice when adding a non passive touch/wheel event listener that DevTools could throw a warning message that would say something like "Hey, if you're not calling preventDefault() later, you might want to pass {passive: true} options. See https://developers.google.com/web/updates/2016/06/passive-event-listeners"
,
Dec 12 2016
,
Dec 14 2016
,
Dec 14 2016
Pavel, this sounds like a violation.
,
Dec 21 2016
Sorry I missed the question earlier. dtapuska@ is the expert here. Personally I think we've realized that outreach on this is largely failing, but that we have a pretty good looking plan to just make touch listeners passive by default. I think we should wait to see how the intervention in issue 639227 goes and then re-evaluate.
,
Dec 22 2016
There's discussion around this for lighthouse here: https://github.com/GoogleChrome/lighthouse/issues/870 I think there is value in encouraging developers to opt in to passive event listeners early, but I'd be fine with waiting until we've seen how the document level passive event listeners intervention goes.
,
Dec 22 2016
I didn't mean to discourage any work you're already planning here! I agree there's value in the warning - I'm just no longer convinced it's worth pushing really hard on and potentially risking warning blindness for. But certainly if it fits in with larger violation/lighthouse plans then yes - let's warn :-)
,
Oct 4 2017
As per comment #7. |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by fbeaufort@chromium.org
, Nov 30 2016