Slick: Prevent Carousel Sliding while drag-scrolling vertically on a touch device

[Defcon0]

Hello,

the title says it all: This should be possible. Is it a bug or already a feature?

Thanks in advance.

Bye Defcon0

[j-kwon]

+1 I have the same issue.

[simeydotme]

set option: verticalSwiping to false ... although I'm super confused, because it defaults to false ;)
https://github.com/kenwheeler/slick#settings

[Defcon0]

I think, you didn't quite get the issue ;-) When I _first_ scroll down in order to get to the stuff at the bottom of the page, and _then_ swipe to the left or the right, it's then possible to scroll _and_ swipe to left and right at the same time, which is ugly.

Edit: I forgot: verticalSwiping is set to false. Still no success.

Edit2: I made a video to make it clear: https://app.box.com/s/778ipwoe0i1t474hla2lmf44b3j88lvk

[simeydotme]

Nice Video! very clear.

[levymetal]

I just started using Slick and I have the same issue. Apart from that, it's incredible!

[levymetal]

I've patched this in my own fork, but I've only tested it in my own personal project. If I get time, I'll try and test out all variations and submit a PR.

Anyway, in the meantime, for anyone who wants a quick fix you can view my changes here: https://github.com/levymetal/slick/commit/36fee6b5ddb51081658e1aa4195a249793f77ddf

EDIT: also, another change here: https://github.com/levymetal/slick/commit/4a5f17e3cc030f2fe36115a3ab4e1525720fa3a6

[Rrrapture]

+1

[Defcon0]

@levymetal: Awesome job :+1: I just integrated your code and it works like a charm. Hopefully, kenwheeler adds this in no time.

[simeydotme]

@levymetal is it possible for you to create a JSFiddle with your source for me to play with ? :)

[adg29]

+1

[ansman]

Any news on this, it's really annoying when using a touch device.

[levymetal]

@ansman Try using my fork here: https://github.com/levymetal/slick.

This works fine for me, but I've not had the time to create a JSFiddle with all possible variations, so I don't wish to submit a PR unless I know it won't break anything.

[colorfool-me]

+1

[hyeyoungkang]

:+1:
Awesome!!
It's a difficult problem to me.. :(
Thank you!!

[jhpmatos]

+1

[floriancapelle]

+1

[leonelcbraz]

+1

[thybzi]

How is about making/merging a pull-request around this?

[theexplay]

+1

[marcelombc]

@kenwheeler, is it possible to add this fix?

[ravigkp]

@kenwheeler Please add this fix , it is so helpful for mobile devices. and this is working like a >kinG-< 👍

[kenwheeler]

Is there a PR open for this?

[ravigkp]

Hey,

Did not got you, come again?

---

Thanks
Ravi

On Sep 20, 2016, at 5:10 PM, Ken Wheeler [email protected] wrote:

Is there a PR open for this?

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/kenwheeler/slick/issues/1252#issuecomment-248277183, or mute the thread https://github.com/notifications/unsubscribe-auth/ACbP-IyngfnjWmpWETEYkOeG1ny_vxkXks5qr8YxgaJpZM4ED1PJ.

[marcelombc]

@levymetal, could you please open a PR?

[leggomuhgreggo]

@kenwheeler looks like PR #862 addresses it.

[levymetal]

@marcelombc It would probably take me 3 weeks or so to get onto it. I wrote that 18 months ago and I'm not sure what's changed in Slick since then, so I'd need to re-assess the fix to ensure it's still compatible. Also, building test cases using each variation/mode can take some time. However, if you'd like to open your own PR, you're more than welcome to use my fix as a base.

[marcelombc]

@levymetal, I open a PR using your fix.
@kenwheeler could you please review the PR

[webegguk]

Having just included Slick (which I've used on many projects) in a predominantly mobile app environment, this is a huge issue. The slick slider is a really robust flexible carousel other than this callback issue on mobile so I don't mean to put it down in any way.
Many thanks @levymetal for the fix, it works great for my situation though it is difficult to see the exact changes as I am using version 1.6 of the plugin which has many changes since your forked version.
I hope this fix gets merged into a new release soon.

[semenets]

+1

[thasmo]

@kenwheeler Please review and merge. :) Thank you!

[thiagocoelho]

I have the same issue, for mobile devices when scrolling up or down. So far the best slider I've found. Praying to see this PR merged soon.. :)

[FullstackJack]

Introduced two years ago and still not merged? Can we please merge this already. It's a major pain.

+100

[leggomuhgreggo]

@FullstackJack looks like you figured it out, but the fix was merged a couple days ago. Gonna close this issue. Thanks everyone.

[m11z]

@leggomuhgreggo slick.min.js file does not include the following fix (#1252), but the source file includes it (not minified slick.js).

[leggomuhgreggo]

The minified file generates with releases, the next one of which will be later this week.

[Amitshaale]

ngx slick carousel touch scroll is not working.
Can anyone help here how to enable touchScroll?