Fixed #1074 where .html() was incorrectly changing the selected value of an option.

davids549 · davids549 · commit 264ffbc73be6 · 2007-11-17T22:09:51.000Z
diff --git a/src/core.js b/src/core.js
@@ -932,7 +932,7 @@ jQuery.extend({
 				var wrap =
 					// option or optgroup
 					!tags.indexOf("<opt") &&
-					[ 1, "<select>", "</select>" ] ||
+					[ 1, "<select multiple='multiple'>", "</select>" ] ||
 					
 					!tags.indexOf("<leg") &&
 					[ 1, "<fieldset>", "</fieldset>" ] ||
diff --git a/test/unit/core.js b/test/unit/core.js
@@ -882,15 +882,19 @@ test("val(String)", function() {
 var scriptorder = 0;
 
 test("html(String)", function() {
-	expect(9);
+	expect(10);
 	var div = $("div");
 	div.html("<b>test</b>");
 	var pass = true;
 	for ( var i = 0; i < div.size(); i++ ) {
-	  if ( div.get(i).childNodes.length == 0 ) pass = false;
+		if ( div.get(i).childNodes.length == 0 ) pass = false;
 	}
 	ok( pass, "Set HTML" );
 
+	$("#main").html("<select/>");
+	$("#main select").html("<option>O1</option><option selected='selected'>O2</option><option>O3</option>");
+	equals( $("#main select").val(), "O2", "Selected option correct" );
+
 	stop();
 
 	$("#main").html('<script type="text/javascript">ok( true, "$().html().evalScripts() Evals Scripts Twice in Firefox, see #975" );</script>');
