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

Added a looping autoplay to the Story page. #305

Merged
merged 1 commit into from
Oct 28, 2018

Conversation

wes977
Copy link
Contributor

@wes977 wes977 commented Oct 27, 2018

I added a story on how to have the autoplay loop when on the last slide.

Description

used the loop prop so that it goes to the first slide after the last slide.
as referenced in issue #288

Motivation and Context

Displays how you can make the carousel infinitely loop-able.

#288 Issue

How Has This Been Tested?

Ran the tests, also went through the different tabs on the story to make sure that they ll act the same post change.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • [X ] New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • [ X] My code follows the code style of this project.
  • [ X] My change requires a change to the documentation.
    • [ X] I have updated the documentation accordingly.
  • [ X] I have included a vue-play example (if this is a new feature)

@quinnlangille
Copy link
Member

Hey @wes977, thanks for this! Since it's just a repo quality update, I'll merge into master right away :octocat:

@quinnlangille quinnlangille merged commit adfbbf5 into SSENSE:master Oct 28, 2018
quinnlangille pushed a commit that referenced this pull request Oct 31, 2018
* Update Carousel.vue

fix carousel bug

* Added a looping autoplay to the Story page. (#305)

* docs(readme): updated README configuration section (#304)
@yashwanth1796
Copy link

@quinnlangille is this new feature add to the last release

@quinnlangille
Copy link
Member

Hey @yashwanth1796, this isn't a new feature - it just adds an example for out already existing loop functionality. You can see the new example in the projects vue-play environment

MartinGerritsen pushed a commit to MartinGerritsen/vue-carousel that referenced this pull request Nov 16, 2018
quinnlangille pushed a commit that referenced this pull request Nov 26, 2018
* Update Carousel.vue

fix carousel bug

* Added a looping autoplay to the Story page. (#305)

* docs(readme): updated README configuration section (#304)
quinnlangille added a commit that referenced this pull request Nov 29, 2018
* feat(standard): adding touchDrag prop to block touch dragging (#296)

* feat(standard): adding touchDrag prop to block dragging on mobile devices

* feat(standard): adding touchDrag prop to block dragging on mobile devices - removing package-lock.json

* adding line-break space to end of file

* chore(build): release for rc1

* v0.16.0-rc1

* Fix carousel failing to lock to remainder slides (#303)

* Update Carousel.vue

fix carousel bug

* Added a looping autoplay to the Story page. (#305)

* docs(readme): updated README configuration section (#304)

* v0.16.0-rc2

* fix: rename event slideClick to slideclick in Slide.vue (#311)

* fix: rename event slideClick to slideclick

* fix(slide): disable tap highlight in Mobile Safari (#326)

* fix(slide): disable tap highlight in Mobile Safari

* feat: add correct calculation for pageCount (#329)

* chore: build
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.

3 participants