Manipulation: simplify html wrappers

markelog · markelog · commit 0ea342a6a6dc · 2015-02-10T23:44:01.000+03:00
Take advantage of html serialization for html wrappers - saves 26 bytes Plus add additional test for "col" element Closes gh-2031 Fixes gh-2002
diff --git a/src/manipulation.js b/src/manipulation.js
@@ -33,9 +33,20 @@ var
 		option: [ 1, "<select multiple='multiple'>", "</select>" ],
 
 		thead: [ 1, "<table>", "</table>" ],
+
+		// Some of the 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
+
+		// Support: Android 2.3
+		// Android browser doesn't auto-insert colgroup
 		col: [ 2, "<table><colgroup>", "</colgroup></table>" ],
-		tr: [ 2, "<table><tbody>", "</tbody></table>" ],
-		td: [ 3, "<table><tbody><tr>", "</tr></tbody></table>" ],
+
+		// Auto-insert "tbody" element
+		tr: [ 2, "<table>", "</table>" ],
+
+		// Auto-insert "tbody" and "tr" elements
+		td: [ 3, "<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" );
