Append colgroup and col elements (#1044)

jquery · Apr 30, 2007 · 5c54a4b8ee5cfea3aff55a1122b187b731319e61 · 5c54a4b
1 parent 74a8e20
commit 5c54a4b8ee5cfea3aff55a1122b187b731319e61
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/src/jquery/coreTest.js b/src/jquery/coreTest.js
@@ -356,7 +356,7 @@ test("wrap(String|Element)", function() {
 });
 
 test("append(String|Element|Array&lt;Element&gt;|jQuery)", function() {
-	expect(14);
+	expect(16);
 	var defaultText = 'Try them out:'
 	var result = $('#first').append('<b>buga</b>');
 	ok( result.text() == defaultText + 'buga', 'Check if text appending works' );
@@ -412,6 +412,11 @@ test("append(String|Element|Array&lt;Element&gt;|jQuery)", function() {
 	$('#select1').append('<OPTION>Test</OPTION>');
 	ok( $('#select1 option:last').text() == "Test", "Appending &lt;OPTION&gt; (all caps)" );
 
+	$('#table').append('<colgroup></colgroup>');
+	ok( $('#table colgroup').length, "Append colgroup" );
+
+	$('#table colgroup').append('<col>');
+	ok( $('#table colgroup col').length, "Append col" );
 });
 
 test("appendTo(String|Element|Array&lt;Element&gt;|jQuery)", function() {

diff --git a/src/jquery/jquery.js b/src/jquery/jquery.js
@@ -1465,7 +1465,7 @@ jQuery.extend({
 					!s.indexOf("<leg") &&
 					[1, "<fieldset>", "</fieldset>"] ||
 
-					(!s.indexOf("<thead") || !s.indexOf("<tbody") || !s.indexOf("<tfoot")) &&
+					(!s.indexOf("<thead") || !s.indexOf("<tbody") || !s.indexOf("<tfoot") || !s.indexOf("<colg")) &&
 					[1, "<table>", "</table>"] ||
 
 					!s.indexOf("<tr") &&
@@ -1475,6 +1475,9 @@ jQuery.extend({
 					(!s.indexOf("<td") || !s.indexOf("<th")) &&
 					[3, "<table><tbody><tr>", "</tr></tbody></table>"] ||
 
+					!s.indexOf("<col") &&
+					[2, "<table><colgroup>", "</colgroup></table>"] ||
+
 					[0,"",""];
 
 				// Go to html and back, then peel off extra wrappers