Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Buggy swiping when using loopedSlides and slidesPerView: 'auto' #2630

Closed
1 of 3 tasks
brandonce opened this issue May 17, 2018 · 7 comments
Closed
1 of 3 tasks

Buggy swiping when using loopedSlides and slidesPerView: 'auto' #2630

brandonce opened this issue May 17, 2018 · 7 comments
Labels
stale

Comments

@brandonce
Copy link

Similar issues have been opened before but were closed without a fix. This one has an updated jsfiddle.

This is a (multiple allowed):

  • bug

  • enhancement

  • feature-discussion (RFC)

  • Swiper Version: at least as early as 3.4.0, and currently 4.2.6.

  • Platform/Target and Browser Versions: macOS, CHROME (all browsers).

  • Live Link or JSFiddle/Codepen or website with issue: https://jsfiddle.net/n88Lhesr/4/

What you did

Click the right arrow button 3 times to get to slide 3, then click the left button.

Expected Behavior

Expected: It goes back to slide 2

Actual Behavior

Actual: Slide 2 vanishes and it stays on slide 3

(I think it's jumping to the duplicated slide 3 that exists to the far left of the "real" slides, which is the true slide index 0). Possible workaround is setting loopedSlides to the total number of slides, but this is not performant if there are numerous or complex slides. I think the index calculations that determine when to jump between duplicate slide N and real slide N need to be examined.
@kochizufan
Copy link
Contributor

I believe this issue is fixed in #2651 PR.

@brandonce
Copy link
Author

brandonce commented Jul 17, 2018 •
edited
Loading

Sorry, but it looks like it is not.

I can still recreate the above steps in this updated jsfiddle (swiper version updated to 4.3.3) : https://jsfiddle.net/n88Lhesr/8/

@stale
Copy link

stale bot commented Oct 15, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Oct 15, 2018
@cgat
Copy link

cgat commented Oct 23, 2018

As noted by @brandonce, this is still an issue. To generalize, if you set loopedSlides to anything less than or equal to half of the amount of slides, you can replicate this behaviour.

@stale stale bot removed the stale label Oct 23, 2018
@cgat
Copy link

cgat commented Oct 23, 2018

I'm using swiper 4.4.1 for what it's worth

@stale
Copy link

stale bot commented Apr 21, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Apr 21, 2019
@stale
Copy link

stale bot commented May 21, 2019

This issue has been automatically closed due to inactivity. If this issue is still actual, please, create the new one.

@stale stale bot closed this as completed May 21, 2019
@lock lock bot added the outdated label Jan 16, 2020
@lock lock bot locked as resolved and limited conversation to collaborators Jan 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
stale
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@kochizufan @cgat @brandonce