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.

Sign up for GitHub

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

Jump to bottom

Missing stop() statement before slideDown() and slideUp() #16

Closed
GeorgMeyer23 opened this issue Mar 8, 2016 · 4 comments
Closed

Missing stop() statement before slideDown() and slideUp() #16

GeorgMeyer23 opened this issue Mar 8, 2016 · 4 comments
Labels
bug

Comments

@GeorgMeyer23
Copy link

What happens:

  • if you click multiple times on the trigger the accordion opens/closes repeatedly.

What should happen:

  • If you click on the trigger the current animation should be stopped and the triggered state should animate.

Hint:

  • Add a stop() statement before using jQuery animation like slideDown() or slideUp().
@ain
Copy link
Contributor

ain commented Mar 8, 2016

Is this a regression?

@GeorgMeyer23
Copy link
Author

I don't know, current behaviour feels buggy :(

@GeorgMeyer23
Copy link
Author

no regression, simple bug.

@GeorgMeyer23 GeorgMeyer23 assigned GeorgMeyer23 and unassigned ain Mar 16, 2016
@GeorgMeyer23 GeorgMeyer23 removed their assignment Mar 16, 2016
@GeorgMeyer23
Copy link
Author

@davleh pls do a quick review, only minor fix, so no PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ain @GeorgMeyer23