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

Keep element structure if it is already in the correct structure. #707

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

Conversation

nvdbleek
Copy link

@nvdbleek nvdbleek commented Mar 8, 2018

There are some frameworks that don't like you to move elements around. By allowing the user to create the viewport and slider nodes themselves, the carousel is usable when you use a framework that doesn't allows you to move elements.

There are some frameworks that don't like you to move elements around. By allowing the user to create the viewport and slider nodes themselves, the carousel is usable when you use a framework that doesn't allows you to move elements.
@desandro
Copy link
Member

desandro commented Mar 9, 2018

Thanks for this PR! I'll have to evaluate this over #381

kapouer pushed a commit to pageboard/flickity that referenced this pull request Apr 16, 2018
There are some frameworks that don't like you to move elements around. By allowing the user to create the viewport and slider nodes themselves, the carousel is usable when you use a framework that doesn't allows you to move elements.

This is metafizzy#707
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants