Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

'moveSlides' > 1 breaks the navigation (fiddle provided) #1224

Open
diegosomar opened this issue Sep 28, 2018 · 1 comment
Open

'moveSlides' > 1 breaks the navigation (fiddle provided) #1224

diegosomar opened this issue Sep 28, 2018 · 1 comment

Comments

@diegosomar
Copy link

I'm developing a calendar, which the days are shown with BX Slider.
Seven days are shown at time.
After click left arrow, previous 7 days are shown. Click right arrow, nexts 7 days are shown.
Quite simple, but....

if 'moveSlides' == 1 then all works.
if 'moveSlides' > 1, the "dots navigation" doesn't changes, which indicates that some calcs are wrong.

Fiddle: https://jsfiddle.net/w3a97zyh/

Expected behavior
In this example, 3 dots (pages) are expected, because I have 21 items in the carousel.

Observed behavior
21 dots (pages) are shown, which indicates that calc ignores the 'moveSlides' option, considering 'moveSlides' == 1.

Did you look for similar issues in Github?

Yes. I looked for "moveSlides" in issue tracker and the questions don't helped me.

Another point

In the same fiddle, I have an option startSlide which doesn't works too. If uncomment the line and change moveSlides to 1, start slide will be DOMINGO 07 out. Note that the last day don't be reached.

There's a reference?

Yes. There's a website with a carousel which shown the days.
https://bit.ly/2xY0myl (I don't know what carousel are in use, but the functionality is the same)

More info

Which browsers/versions does it happen on? Google Chrome 69.0.3497.100 (Official Version) 64 bits
Which jQuery/BX Slider version are you using? jQuery 1.12.4 / BX Slider 4.2.15
Did this work before? I don't know

@SamDeNies
Copy link

This appears to be a regression beginning in 4.2.13.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants