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

Fixes #2572 #2581

Closed
wants to merge 1 commit into from
Closed

Fixes #2572 #2581

wants to merge 1 commit into from

Conversation

malacalypse
Copy link
Contributor

Since the input state is toggled by set.checked() the default behaviour of this click does not need to happen.

This fixes the multiple trigger bug with labels and input event bubbling, while permitting input state change and bindings to still work.

@jlukic jlukic added this to the 2.0.3 milestone Jul 10, 2015
@jlukic
Copy link
Member

jlukic commented Jul 10, 2015

Thanks for fixing this bug. In future, please submit PR against next branch this way I can merge your commits and you can get credit. (Check readme for details).

@jlukic jlukic closed this Jul 10, 2015
jlukic added a commit that referenced this pull request Jul 10, 2015
@malacalypse
Copy link
Contributor Author

Roger that. I'll get the PR right next time. Thank you for the credit!

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.

2 participants