Yes! A second click on the dropdown arrow _should_ collapse the dropd…

…own box. Closes gh-1
harvesthq · Jul 18, 2011 · ed9e82a · ed9e82a
1 parent 137afac
commit ed9e82a
Show file tree

Hide file tree

Showing 4 changed files with 30 additions and 4 deletions.
diff --git a/chosen/chosen.jquery.js b/chosen/chosen.jquery.js
diff --git a/chosen/chosen.proto.js b/chosen/chosen.proto.js
diff --git a/coffee/chosen.jquery.coffee b/coffee/chosen.jquery.coffee
@@ -119,7 +119,7 @@ class Chosen
         this.results_show()
       else if not @is_multiple and evt and ($(evt.target) is @selected_item || $(evt.target).parents("a.chzn-single").length)
         evt.preventDefault()
-        this.results_show()
+        this.results_toggle()
 
       this.activate_field()
     else
@@ -248,6 +248,12 @@ class Chosen
     @result_highlight.removeClass "highlighted" if @result_highlight
     @result_highlight = null
 
+  results_toggle: ->
+    if @results_showing
+      this.results_hide()
+    else
+      this.results_show()
+
   results_show: ->
     if not @is_multiple
       @selected_item.addClass "chzn-single-with-drop"

diff --git a/coffee/chosen.proto.coffee b/coffee/chosen.proto.coffee
@@ -114,7 +114,7 @@ class Chosen
         document.observe "click", @click_test_action
         this.results_show()
       else if not @is_multiple and evt and (evt.target is @selected_item || evt.target.up("a.chzn-single"))
-        this.results_show()
+        this.results_toggle()
 
       this.activate_field()
     else
@@ -242,6 +242,12 @@ class Chosen
     @result_highlight.removeClassName('highlighted') if @result_highlight
     @result_highlight = null
 
+  results_toggle: ->
+    if @results_showing
+      this.results_hide()
+    else
+      this.results_show()
+
   results_show: ->
     if not @is_multiple
       @selected_item.addClassName('chzn-single-with-drop')