fix: ignored attempt to cancel a touchmove event with cancelable=false

[hueldera]

Description

Fix the [Intervention]Ignored attempt to cancel a touchmove event with cancelable=false, for example because scrolling is in progress and cannot be interrupted. console error when scrolling with touch on mobile.

Type of Change

• [x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

• Add height: 200vh and align-items: center to Drag Multiple Slides example in Storybook
• Scroll with the cursor over the slide card on the Surface Duo device in DevTools (It may not give an error in other device sizes in this example because of the buttons covering the slide).

Checklist: (Feel free to delete this section upon completion)

• [x] My code follows the style guidelines of this project (I have run yarn lint)
• [ ] I have added tests that prove my fix is effective or that my feature works
• [x] New and existing unit tests pass locally with my changes (I have run yarn test and yarn test-e2e)
• [x] I have performed a self-review of my own code
• [ ] I have commented my code, particularly in hard-to-understand areas
• [ ] I have made corresponding changes to the documentation
• [ ] For any breaking API changes, I have updated the type definitions in index.d.ts
• [x] My changes generate no new warnings
• [ ] Any dependent changes have been merged and published in downstream modules

[vercel[bot]]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated
nuka-carousel	✅ Ready (Inspect)	Visit Preview	Jun 14, 2022 at 6:17PM (UTC)

[ValGeorgiev]

Hey @hueldera, thank you for the PR. Which version of the nuka-carousel are you using?

[hueldera]

Hey @hueldera, thank you for the PR. Which version of the nuka-carousel are you using?

Hi @ValGeorgiev, I tested with the main branch (v5.1.1)

[ValGeorgiev]

Hey @hueldera, thank you for the PR. Which version of the nuka-carousel are you using?

Hi @ValGeorgiev, I tested with the main branch (v5.1.1)

thats weird cause your changes are on the legacy src/ folder which will be deprecated soon. And I am not sure if this will fix the issue.

[hueldera]

@ValGeorgiev Sorry, I completely ignored that the versions coexist, so in V5 if you enable touch action, we don't have this error. It turns out that this PR updates the current V4 version, in production I'm still on it for now.