fix bug #2310 #2387

Closed
wants to merge 7 commits into
from
Commits on Jul 30, 2012
  1. fix bug #2310

    Cristian Carlesso committed Jul 30, 2012
    IE7 clone node
    (http://msdn.microsoft.com/en-us/library/ie/ms536365(v=vs.85).aspx)
    have problem when cloning element with id attribute stetted.
  2. added tests

    Cristian Carlesso committed Jul 30, 2012
    added tests, fixed some spacing, precedence issues
  3. adding missing semicolon, and a little comment

    Cristian Carlesso committed Jul 30, 2012
    adding missing semicolons in the spec, and a little comment on the fix.
Commits on Aug 14, 2012
  1. After further tests, seems like that the only way to avoid the proble…

    Cristian Carlesso committed Aug 14, 2012
    …m is to not use clone node at all
    
    After further tests, seems like that the only way to avoid the problem
    is to not use clone node at all.
    
    So, I wrote a clone node function using outerHTML. need further
    testing, to be sure everything is ok, but since it passes regression
    I'll commit it.
Commits on Aug 19, 2012
  1. add a little more test

    Cristian Carlesso committed Aug 19, 2012
    more test added
Commits on Aug 20, 2012
  1. adding another failing test

    Cristian Carlesso committed Aug 20, 2012
    add a failing test.
    TODO: split tests
  2. quick fix to pass the failing test

    Cristian Carlesso committed Aug 20, 2012
    quick fix to pass the failing test