> That IE7 crashes is related to
> https://mootools.lighthouseapp.com/projects/2706/tickets/1196-cloning-element-twice-crashes-ie7

I  have  been  looking into this, esp. since my patch for #1058 led to
the regression. I am very, very close to a solution.

Basically,  we  need to never clone an element while it has a non-null
id.  We  can  cache the id in a local var, set the element's id to the
null  attributenode  (in  the  original  patch,  this  was done on the
*clone*),  clone the element, then restore the id with anode.nodeValue
= <cached_value>.

I  have  tested  this  sequence  with  IE  7 and I've been able to set
dynamic  ids  on  the clone *and* dispose of both originals and clones
without  error/crash.  However, I need to go back to IE 6 with this as
well.

More in Lighthouse next couple of days....

-- S.

Reply via email to