Check if event is cancelable before calling preventDefault

[crenshaw-dev]

This uses MDN's recommendation to avoid preventDefault when it shouldn't be used:
https://developer.mozilla.org/en-US/docs/Web/API/Event/cancelable

Sorry for the messy diff. I think Docker made the whitespace changes? Let me know if you'd like me to clean it up and submit another PR.

[crenshaw-dev]

Just discovered this doesn't completely clean up the console. But I think it's a step in the right direction.

[FlorianKoerner]

Clean Diff: https://github.com/ganlanyuan/tiny-slider/pull/357/files?utf8=%E2%9C%93&diff=split&w=1

[eric-blue]

Clean Diff: https://github.com/ganlanyuan/tiny-slider/pull/357/files?utf8=%E2%9C%93&diff=split&w=1

Can we have this merged?

[Eduruiz]

This solves the issue for me.

[FredericBriolet]

Adding preventScrollOnTouch: 'auto' to the slider's constructor removed reccurent preventDefault errors for me.

However a new wild one appears sometimes :
tiny-slider.js:2558 [Intervention] Ignored attempt to cancel a touchmove event with cancelable=false, for example because scrolling is in progress and cannot be interrupted.

So yes, check if event is cancellable in OnPanMove would be great 👍

[SuperSchek]

+1. Could really use this fix.

[crenshaw-dev]

I'm not sure how to resolve this. Since these are built files, should I just choose my version? Super difficult to read the diff.

[drjgray]

Any update on this issue?

[artilishes]

Any news on this issue?

[demyan1411]

?

[savchukm]

@mac9416 Could you please clean up code changes? I hope it gets merged soon.

[panec]

Updated PR: #437