github/jquery
1
1
Fork 0
mirror of https://github.com/jquery/jquery.git synced 2026-02-18 14:04:25 -05:00
Code Issues Packages Projects Releases Wiki Activity

Manipulation: Restrict the tbody search to child nodes

Browse Source 
For performance, use a querySelectorAll path instead of Javascript iteration.
http://codepen.io/anon/pen/vywJjx?editors=1010

Fixes gh-3439
Closes gh-3463
This commit is contained in:
Richard Gibson
2017-01-09 11:37:12 -08:00
committed by GitHub
parent 9d822bc1c1
commit efdb8a46e4
2 changed files with 22 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
test/unit/manipulation.js
View File

@@ -2730,6 +2730,26 @@ QUnit.test( "Make sure col element is appended correctly", function( assert ) {
assert.strictEqual( table.find( "td" ).width(), 150 );
} );
QUnit.test( "Make sure tr is not appended to the wrong tbody (gh-3439)", function( assert ) {
assert.expect( 1 );
var htmlOut,
htmlIn =
"<thead><tr><td>" +
"<table><tbody><tr><td>nested</td></tr></tbody></table>" +
"</td></tr></thead>",
newRow = "<tr><td>added</td></tr>",
htmlExpected = htmlIn.replace( "</thead>", "</thead>" + newRow ),
table = supportjQuery( "<table/>" ).html( htmlIn ).appendTo( "#qunit-fixture" )[ 0 ];
jQuery( table ).append( newRow );
// Lowercase and replace spaces to remove possible browser inconsistencies
htmlOut = table.innerHTML.toLowerCase().replace( /\s/g, "" );
assert.strictEqual( htmlOut, htmlExpected );
} );
QUnit.test( "Insert script with data-URI (gh-1887)", 1, function( assert ) {
Globals.register( "testFoo" );
Globals.register( "testSrcFoo" );