kswedberg / jquery-carousel-lite

A jQuery carousel plugin based on jCarouselLite by Ganeshji Marwaha
MIT License
177 stars 59 forks source link

[feature] multiple activeClass options #14

Closed m5o closed 12 years ago

m5o commented 12 years ago

Hey Karl!

This is more a feature request as an issue.

The activeClass option is a great feature.

In your demo there are always two items visible. In my opinion the 'active'-class should on all visible navigation items. So the user get a visual feedback which items he is currently viewing and which are hidden. Or maybe a separate helper class eg. 'visible'

In our project we have "dot"-style navigation. (visible: 4, scroll: 4) seen on SwipeJs [1] or Apple [2].

What do you think about this feature?

Thx, m5o

[1] http://swipejs.com/ [2] http://www.apple.com/macmini/

kswedberg commented 12 years ago

thanks a lot for the request. I like the general idea of supporting pagination better, but I'm not sure how it should be implemented. Both of the examples you link to only have one item visible at a time. this makes it really easy to use "dot"-style navigation without any change in the plugin. I'm wondering, though, what you think it should look when multiple items are visible at a time. Should there be only one "dot" per group of visible items? Or should there be a dot for each item and all 4 dots representing the "visible" items be "active"?

m5o commented 12 years ago

There should be a "dot" or numbered list (seen in your demo) for each item and all active items are highlighted with a separate helper class or the current active class