Hi,
I've added a new patch to the ticket based on Volker's patch. Can you please
confirm that it does indeed fix the problem?http://dev.jquery.com/ticket/1143
No it doesn't work as it doesn't travers all node recursively. I've
written a new patch, please take a look at
Hi,
On 24 Mai, 22:39, John Resig [EMAIL PROTECTED] wrote:
$(...).find(*).each(function(){ delete this.mergeNum; });
This won't work in Internet Explorer.
Having the value come up as mergeNum=null tells me that there's
something very wrong at play here. What exactly are you using to
I think mergeNum won't be added everytime, it gets only added if you
need to run an uniqueness test (when find() was called). I even think
unique() can be removed when merge gets changed a bit. I've some
ideas, at the end it should be faster and have a smaller file size.
Cu,
Volker.
On 24
() uses for ( var i = 0; second[i]; i++ )
for loops, all other cases where such a loop was used, it's like for
( var i = 0, fl = first.length; i fl; i++ ). If the way merge()
does it work a few bits could be saved.
Cu,
Volker.
PS: Sorry for multiposting.
On 24 Mai, 22:46, Volker Mische [EMAIL
4 matches
Mail list logo