search

basilio / responsiveCarousel

Simple carousel
MIT License
113 stars 64 forks source link

Visible items #4

Closed dlzi closed 11 years ago

dlzi commented 11 years ago

Hey Basilio,

Let's say that I have 4 items in my carousel and I want to made all of them visibile. So, I set the option to "visible: 4". But, If I do that, my first item will not appear. It turned out that if I want to have 4 visible items, my carousel has to have 5 items. The carousel visible items should always be minus one of the total items.

Is it a bug or is it the way that carousel works?

Thank you.

basilio commented 11 years ago

Hi Daniel, Sorry for the delay, i'm a little busy with my work this week :( I understand perfectly this bug. This has two branches:

  1. Create conditions so that no logical errors in variables (easy)
  2. Modify the core behavior for your case (humm)

I will work on that ASAP. Thanks again :+1:

dlzi commented 11 years ago

Hello!

No worries, take your time.

Thank you.

basilio commented 11 years ago

Daniel, I solved this bug finally :) Thanks

dlzi commented 11 years ago

Great news!

Thank you!