Diego, that patch makes the error go away because the selected node
now has a parentNode, but it's the wrong parentNode--it's the document
fragment instead of the original document. So, the element gets left
behind in the document fragment instead of being inserted back into
the document.

It looks like the original code has some problems as well.

for ( var i = 0, l = this.length; i < l; i++ )
   callback.call( root(this[i], first),
     this.length > 1 || i > 0 ?
        fragment.cloneNode(true) : fragment );

The expression "this.length > 1 || i > 0" will always be true whenever
this.length>1 so what is the purpose of the i>0? I *think* the intent
was "this.length > 1 && i > 0" with the idea that the first insert
would be the original nodes and the subsequent ones would be clones of
that. Even with that change it still causes the problem reported in
#4087 though.



--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"jQuery Development" group.
To post to this group, send email to jquery-dev@googlegroups.com
To unsubscribe from this group, send email to 
jquery-dev+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/jquery-dev?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to