search

kenwheeler / slick

the last carousel you'll ever need
kenwheeler.github.io/slick
MIT License
28.54k stars 5.89k forks source link

Vertical mode should allow to pan vertically, not horizontally #654

Closed nimaen closed 10 years ago

nimaen commented 10 years ago

Hi,

Something I've found while willing to use your awesome carousel. When the vertical mode is activated, you cannot drag the content vertically, only horizontally. Seems like a totally weird UX to me, don't you think ?

Are you aware of this issue, and if yes, do you have a little snippet that would allow you to drag the content vertically ?

kenwheeler commented 10 years ago

Vertical scrolling should be reserved for scrolling the page. This was a purposeful decision, and the weird UX would be hijacking the user's vertical scrolling in order to scroll a carousel.

kenwheeler commented 10 years ago

Try this on a phone: http://bxslider.com/examples/vertical-slideshow

zachleat commented 10 years ago

Gotta +1 to @kenwheeler here. It’s an antipattern hijack scrolling for other purposes, especially on small screens and viewports!

nimaen commented 10 years ago

I agree with you on one purpose : when it applies to classical layouts. When building fullscreen web apps with complex layouts, you will therefore handle scrolls / navigation on your own way.

The strange behavior I was pointing out was that the carousel were sliding vertically when swiping horizontally. And it feels definitely weird.

nimaen commented 10 years ago

I might be the only finding it weird though :)

zachleat commented 10 years ago

I think that anything that lives on the web and doesn’t use Responsive Web Design is an anti-pattern too :)

nimaen commented 10 years ago

Seems like there is no possible discussion :) Your carousel is still good for all the other stuff.

That's just sad.

kenwheeler commented 10 years ago

@Nimaen I don't mean to close the discussion down. If you search the issues on this (in the first place), you'll find the way to make it work.

That said, I don't think it belongs in core. Vertical, IMO, shouldn't even be in core.

If you want to make a bad UX decision, go ahead, but I don't want to be the guy who helped you do it.

kenwheeler commented 10 years ago

@zachleat what do you mean?

zachleat commented 10 years ago

@kenwheeler mostly just responding to

When building fullscreen web apps with complex layouts

which seems to imply that there are a class of web apps/sites shouldn’t work on smallscreen (mobile) devices, which I disagree with wholeheartedly. If it’s on the web it should work.

kenwheeler commented 10 years ago

@zachleat +1