Manipulation: simplify html wrappers

markelog · markelog · commit 9fc377b62b20 · 2015-01-26T04:32:22.000+03:00
Take advantage of html serialization for html wrappers - saves 26 bytes
Plus add additional test for "col" element
diff --git a/src/manipulation.js b/src/manipulation.js
@@ -33,9 +33,27 @@ var
 		option: [ 1, "<select multiple='multiple'>", "</select>" ],
 
 		thead: [ 1, "<table>", "</table>" ],
-		col: [ 2, "<table><colgroup>", "</colgroup></table>" ],
-		tr: [ 2, "<table><tbody>", "</tbody></table>" ],
-		td: [ 3, "<table><tbody><tr>", "</tr></tbody></table>" ],
+
+		// Following wrappers are not fully defined because
+		// their parent elements (except for "table" element) could be omitted
+		// since browser parsers are smart enough to auto-insert them
+		col: [
+
+			2,
+			// Support: Android 2.3
+			// Android browser doesn't auto-inserts colgroup element
+			"<table><colgroup>", "</colgroup></table>"
+		],
+
+		tr: [
+			2, // - auto-inserts "tbody" element
+			"<table>", "</table>"
+		],
+
+		td: [
+			3, // - auto-inserts "tbody" and "tr" elements
+			"<table>", "</table>"
+		],
 
 		_default: [ 0, "", "" ]
 	};
diff --git a/test/unit/manipulation.js b/test/unit/manipulation.js
@@ -2452,6 +2452,18 @@ test( "Validate creation of multiple quantities of certain elements (#13818)", 4
 	});
 });
 
+test( "Make sure col element is appended correctly", function() {
+	expect( 1 );
+
+	var table = jQuery( "<table cellpadding='0'><tr><td>test</td></tr></table>" );
+
+	jQuery( table ).appendTo( "#qunit-fixture" );
+
+	jQuery( "<col width='150'/>" ).prependTo( table );
+
+	strictEqual( table.find( "td" ).width(), 150 );
+});
+
 asyncTest( "Insert script with data-URI (gh-1887)", 1, function() {
 	Globals.register( "testFoo" );
 	Globals.register( "testSrcFoo" );
