single_deselect change event not fired (Jquery) #700

Closed
paulmach opened this Issue Jun 24, 2012 · 4 comments

Comments

Projects
None yet
5 participants
@paulmach

First select an item, then clear it using the single deselect feature. When you immediately select the same item again, the change event is not fired.

After debugging this is because this.current_value is not cleared when the single_deselet 'X' is clicked.

For me this was fixed by adding the following to Chosen.prototype.results_reset(), line 706 of chosen.jquery.js of the current downloadable version.

this.current_value = '';

I hope this helps someone.

@ouvanous

This comment has been minimized.

Show comment Hide comment
@ouvanous

ouvanous Jun 27, 2012

Thanks for the tip !

Thanks for the tip !

@krosaen

This comment has been minimized.

Show comment Hide comment
@krosaen

krosaen Jul 12, 2012

Glad to see someone else has found this too. For anyone looking to easily reproduce, here's a jsfiddle:

http://jsfiddle.net/krosaen/hyJEd/

select option 1, clear it, and then select it again - notice the change event isn't fired.

krosaen commented Jul 12, 2012

Glad to see someone else has found this too. For anyone looking to easily reproduce, here's a jsfiddle:

http://jsfiddle.net/krosaen/hyJEd/

select option 1, clear it, and then select it again - notice the change event isn't fired.

@crazymykl

This comment has been minimized.

Show comment Hide comment
@crazymykl

crazymykl Aug 22, 2012

Contributor

This is still happening.

Contributor

crazymykl commented Aug 22, 2012

This is still happening.

crazymykl added a commit to crazymykl/chosen that referenced this issue Aug 22, 2012

@crazymykl crazymykl referenced this issue Aug 30, 2012

Closed

bugfix #792

@pfiller pfiller closed this in 04f6b87 Sep 3, 2012

@pfiller

This comment has been minimized.

Show comment Hide comment
@pfiller

pfiller Sep 3, 2012

Contributor

Fixed in the latest version of Chosen.

Contributor

pfiller commented Sep 3, 2012

Fixed in the latest version of Chosen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment