-
Notifications
You must be signed in to change notification settings - Fork 56
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
BO: Fixed position with new slide and remove check on non present input position #23
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, I wonder if we won't have a broken with this PR. It seems this method is called when we try to reorder slides, how will the position be taken in account?
You're right, this method need to stay public. And about new slide added, get the next position with my commit ? |
Ping @Quetzacoalt91 to @beyondsagency question ;) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I totally missed it, sorry. It now looks good to me.
Ping @beyondsagency for answer ;) |
OK can we merge ? |
@beyondsagency Sorry, I forgot to send QA ;) |
Thanks @beyondsagency |
This is a fix for ps_imageslider whe we add a new slide the position is not good and we have function getNextPosition() not use, so use it :)
Beyonds, Digital Agency