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

Remove pending_destroy_click variable. #1149

Merged
merged 2 commits into from
Apr 18, 2013

Conversation

pfiller
Copy link
Contributor

@pfiller pfiller commented Apr 18, 2013

Event bubbling on a destroy choice click stopped with #1142 which means the pending destroy click never gets set to false which means that the pending_destroy_click is unecessary.

Less code, easier to read and fixes #706

Event bubbling on a destroy choice click stopped with #1142 which
means the pending destroy click never gets set to false which means
that the pending_destroy_click is unecessary.

Fixes #706
@kenearley
Copy link

Very nice. Works well. :shipit:

Conflicts:
	chosen/chosen.jquery.js
	chosen/chosen.jquery.min.js
	chosen/chosen.proto.js
	chosen/chosen.proto.min.js
pfiller added a commit that referenced this pull request Apr 18, 2013
…res-two-clicks

Remove pending_destroy_click variable.
@pfiller pfiller merged commit adc8cc0 into master Apr 18, 2013
@pfiller pfiller deleted the fix-delete-last-element-requires-two-clicks branch April 18, 2013 19:37
pfiller added a commit that referenced this pull request Apr 19, 2013
Better container reference #1151
Fix issue when destroying last choice #1149
Use selectedIndex for tracking change events. #1147
Better IE Disabling #1144
Dropdown use css positioning for top position #1143
Fix bug when deleting last item #1142
Percentage Width Support #1141
CSS Centering Fix #1120
Use New CoffeeScript #1125
Chosen License #1111
Update placeholder text with ``liszt:updated`` #1107
Fix strange event bubbling #1055
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.

when I remove the last
2 participants