I cannot duplicate problem with 7d8f735 - this code works in IE. This…

… gets rid of an extra loop.
NicolasCARPi · Jul 22, 2011 · 39b77de · 39b77de
1 parent 0ca8b7f
commit 39b77de
Showing 1 changed file with 6 additions and 8 deletions.
diff --git a/jquery.jeditable.js b/jquery.jeditable.js
@@ -510,15 +510,13 @@
                             continue;
                         } 
                         var option = $('<option />').val(key).append(json[key]);
-                        $('select', this).append(option);    
-                    }                    
-                    /* Loop option again to set selected. IE needed this... */ 
-                    $('select', this).children().each(function() {
-                        if ($(this).val() == json['selected'] || 
-                            $(this).text() == $.trim(original.revert)) {
-                                $(this).attr('selected', 'selected');
+                        if (key == json['selected'] ||
+                            json[key] == $.trim(original.revert)) {
+                                $(option).attr('selected', 'selected');
                         }
-                    });
+                        $('select', this).append(option);
+                    }
+
                     /* Submit on change if no submit button defined. */
                     if (!settings.submit) {
                         var form = this;