jQuery: The Write Less, Do More JavaScript Library

Ticket #2184 (closed bug: fixed)

Opened 4 months ago

Last modified 4 months ago

clone won't work with THEAD, TFOOT, TBODY, TR and TD in MSIE

Reported by: kowcik Assigned to: anonymous
Type: bug Priority: major
Milestone: 1.2.3 Component: core
Version: 1.2.2 Keywords:
Cc: Needs: Review

Description

Beacuse clone uses innerHTML if the detected browser is MS Internet Explorer, cloning won't work with these nodes. See attached test case.

Attachments

clone.html (1.1 kB) - added by kowcik 4 months ago.
Script will attempt to clone table elements using jQuery methods and native DOM. Results will be alerted for comparison.
2184.diff (1.8 kB) - added by davidserduke 4 months ago.
possible patch
fix-for-1.2.2-clone.diff (1.4 kB) - added by davojan 4 months ago.
Another possible solution. It doesn't use clean() and should be faster, but it's bigger.

Change History

Changed 4 months ago by kowcik

Script will attempt to clone table elements using jQuery methods and native DOM. Results will be alerted for comparison.

Changed 4 months ago by kowcik

I've just checked with previously reported bugs, it is a duplicate of this bug: http://dev.jquery.com/ticket/2169, but my test-case is more extensive. I've went thru these test with 1.2.1 and all tested elements worked , so this is a regression.

Changed 4 months ago by davidserduke

possible patch

Changed 4 months ago by davidserduke

See also #2169 which has more information.

Changed 4 months ago by davojan

Another possible solution. It doesn't use clean() and should be faster, but it's bigger.

Changed 4 months ago by davidserduke

Someone told me that <option> also is having problems. I'll check with John and Brandon to get their thoughts.

Changed 4 months ago by davidserduke

  • status changed from new to closed
  • resolution set to fixed

Fixed in [4513].

Note: See TracTickets for help on using tickets.