deac/jquery
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Skip around inserting a fragment when possible (insert the node directly).

Browse source
This commit is contained in:
jeresig 2010-01-28 17:18:27 -05:00
parent b8076a914b
commit 388a00fe91
1 changed files with 13 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
src/manipulation.js
View file

@ -296,7 +296,7 @@ jQuery.fn.extend({
}, },
domManip: function( args, table, callback ) { domManip: function( args, table, callback ) {
var results, first, value = args[0], scripts = []; var results, first, value = args[0], scripts = [], fragment;
// We can't cloneNode fragments that contain checked, in WebKit // We can't cloneNode fragments that contain checked, in WebKit
if ( !jQuery.support.checkClone && arguments.length === 3 && typeof value === "string" && rchecked.test( value ) ) { if ( !jQuery.support.checkClone && arguments.length === 3 && typeof value === "string" && rchecked.test( value ) ) {
@ -321,7 +321,13 @@ jQuery.fn.extend({
results = buildFragment( args, this, scripts ); results = buildFragment( args, this, scripts );
} }
first = results.fragment.firstChild; fragment = results.fragment;
if ( fragment.childNodes.length === 1 ) {
first = fragment = fragment.firstChild;
} else {
first = fragment.firstChild;
}
if ( first ) { if ( first ) {
table = table && jQuery.nodeName( first, "tr" ); table = table && jQuery.nodeName( first, "tr" );
@ -331,14 +337,14 @@ jQuery.fn.extend({
table ? table ?
root(this[i], first) : root(this[i], first) :
this[i], this[i],
results.cacheable || this.length > 1 || i > 0 ? i > 0 || results.cacheable || this.length > 1 ?
results.fragment.cloneNode(true) : fragment.cloneNode(true) :
results.fragment fragment
); );
} }
} }
if ( scripts ) { if ( scripts.length ) {
jQuery.each( scripts, evalScript ); jQuery.each( scripts, evalScript );
} }
} }