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

Correctly center slider #1901

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

Conversation

niaccurshi
Copy link

Ensure the slides are centered if slideCount is less than slidesToShow.

(This is only the unminified version, haven't the time to do anything other than fix this issue!)

Ensure the slides are centered if slideCount is less than slidesToShow
@simeydotme
Copy link
Collaborator

@niaccurshi could you please provide a JS fiddle for before and one for after? Then I can test and merge.

@niaccurshi
Copy link
Author

Sorry that I never got back to this, as I mentioned in my other comment I was a little rushed/busy and so only wanted to provide something to highlight a route forward. Is there anything else you need from me on this at this stage? I'm afraid at this point I don't know if I can go back to provide a before or after on JSFiddle

@stevenorell
Copy link

Any updates on the PR?

@stevenorell-dc
Copy link

...never mind, I see that the issue was fixed and merged in #2118.

@matsaman
Copy link

wfm, thanks

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

Successfully merging this pull request may close these issues.

6 participants