Merge pull request #50 from michaelperrin/fix/ie8-compatibility

Fix IE7 and IE8 compatibility
meltingice · Jan 9, 2013 · de37a51 · de37a51
2 parents 8ac190e + ea7f934
commit de37a51
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 7 deletions.
diff --git a/lib/ajax-chosen.js b/lib/ajax-chosen.js
diff --git a/lib/ajax-chosen.min.js b/lib/ajax-chosen.min.js
diff --git a/src/ajax-chosen.coffee b/src/ajax-chosen.coffee
@@ -93,9 +93,14 @@ do ($ = jQuery) ->
             # value => text pairs to inject as <option> elements.
             items = callback data
 
+
+            nbItems = 0
+
             # Iterate through the given data and inject the <option> elements into
             # the DOM if it doesn't exist in the selector already
             $.each items, (i, element) ->
+              nbItems++
+
               if element.group
                 group = select.find("optgroup[label='#{element.text}']")
                 group = $("<optgroup />") unless group.size()
@@ -126,8 +131,8 @@ do ($ = jQuery) ->
                     .attr('value', value)
                     .html(text)
                     .appendTo(select)
-                
-            if Object.keys(items).length
+
+            if nbItems
               # Tell chosen that the contents of the <select> input have been updated
               # This makes chosen update its internal list of the input data.
               select.trigger("liszt:updated")