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.

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

fix changeSlider for infinity false case #1950

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix changeSlider for infinity false case #1950

wants to merge 1 commit into from

Conversation

VadimSvirdoff
Copy link

Fix bug issue #1949 .

@shrutipant7
Copy link

Please fix this issue soon.

@vadimgordenya
Copy link

@VadimSvirdoff The same issue will be for "next" arrow too. Can you add a fix for this case to your pull request?

@Faraz1995
Copy link

please fix this issue ,
we need this bug to solve
for god sake

@VadimSvirdoff
Copy link
Author

VadimSvirdoff commented Jan 15, 2021

@akiran ready to merge

@Faraz1995
Copy link

we are waiting for this merge

@jorrit
Copy link
Contributor

jorrit commented May 7, 2021

I don't think this patch will be merged as it is, as you are just commenting some code. Are you sure the code can be removed without side effects? Have you checked what the purpose of this code was? If it really can be removed, you should just remove it and not comment it.

@jorrit
Copy link
Contributor

jorrit commented May 7, 2021

It got broken in commit 1c3d70c which was part of PR #1873. The problem is that targetSlide is not initialized to the initialSlide in InnerSlider. I propose another change in #2029.

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

Successfully merging this pull request may close these issues.

None yet

5 participants