"Added non-passive event listener" Chrome violation

When opening photoswipe there are a couple of violations in Chrome:

Added non-passive event listener to a scroll-blocking 'wheel' event. Consider marking event handler as 'passive' to make the page more responsive. See https://www.chromestatus.com/feature/5745543795965952

This repeats for the mousewheel event too

I believe it originates from this line: https://github.com/dimsemenov/PhotoSwipe/blob/703c415b1b97e8983fff217644888062f07d308b/src/js/framework-bridge.js#L15

I don’t think this is a duplicate issue but it may be related to https://github.com/dimsemenov/PhotoSwipe/issues/1217

As it doesn’t call preventDefault it seems worth it to add this in, thoughts? It’d just be changing it to:

if(type[i]) {
  target[methodName]( type[i], listener, ['mousewheel', 'wheel'].indexOf(type[i]) === -1 ? false :
 {passive: true});
}

[andreszs]

This problem persists with 4.1.3 and Chrome 80.0.3987.116.

To prevent this warning, I replaced "wheel mousewheel DOMMouseScroll" with just "DOMMouseScroll". There is no mouse wheel zoom now, but there is a zoom button in the toolbar. And this is being used in a Cordova app, so there is no mouse wheel available anyway.

Enabling/disabling the mousewheel should be configurable from the photoswipe Options, specially considering that 80% of web traffic (and growing) comes from mobile devices, where there’s no mousewheel.

[mhluska]

I’m also seeing these with 4.1.3 on Chrome 79.0.3945.130