Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

Just click quickly among the indicators to break the carousel #3729

Closed
chogarcia opened this issue May 28, 2015 · 1 comment
Closed

Just click quickly among the indicators to break the carousel #3729

chogarcia opened this issue May 28, 2015 · 1 comment
Labels
component: carousel type: bug
Milestone
0.13.1 (npm)

Comments

@chogarcia
Copy link

So, you just click quickly among the indicators to break the carousel.

Any clue?
@rvanbaalen rvanbaalen added this to the Backlog milestone May 28, 2015
@rvanbaalen
Copy link
Contributor

Bug confirmed. Clicking aggressively on the carousel indicators temporarily breaks the functionality.

@wesleycho wesleycho mentioned this issue Jun 4, 2015
Closed
@wesleycho wesleycho modified the milestones: 0.13.1 (Performance), Backlog Jun 4, 2015
bleggett pushed a commit to bleggett/bootstrap that referenced this issue Jun 11, 2015
@wesleycho @bleggett
fix(carousel): disable transition until animation completes
1b9f41d 
- Force the carousel indicator to not select the slide if the animation has not completed
- Factor out `goNext` function to not redefine on each execution of `select`

chore(carousel): move `$currentTransition` check to `select` method

Fixes angular-ui#3729
Closes angular-ui#3757
bleggett pushed a commit to bleggett/bootstrap that referenced this issue Jun 11, 2015
@wesleycho @bleggett
fix(carousel): disable transition until animation completes
eb8562f 
- Force the carousel indicator to not select the slide if the animation has not completed
- Factor out `goNext` function to not redefine on each execution of `select`

chore(carousel): move `$currentTransition` check to `select` method

Fixes angular-ui#3729
Closes angular-ui#3757
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
component: carousel type: bug
Projects
None yet
Milestone
0.13.1 (npm)
Development

No branches or pull requests
3 participants
@rvanbaalen @wesleycho @chogarcia