loryjs / lory

☀ Touch enabled minimalistic slider written in vanilla JavaScript.
http://loryjs.github.io/lory/
MIT License
2.28k stars 243 forks source link

Unable to preventDefault inside passive event listener invocation. #727

Closed oneezy closed 6 years ago

oneezy commented 6 years ago

When using enableMouseEvents: true + Google Chrome's Dev Tools "Device Toolbar" and apply a "drag" with the mouse, this bug pops up in the console. It's odd because the bug doesn't actually show up unless you've got the device toolbar opened.

image

ram-you commented 6 years ago

+1, only when "touch" is enabled. Have to make test if device touch is enabled.

ram-you commented 6 years ago

A link to help undestand the problem: https://developers.google.com/web/updates/2017/01/scrolling-intervention

meandmax commented 6 years ago

Can be closed because prevent default was removed because of other issues and because it is not needed.

Thanks for the issue.