Bug Tracker

Opened 13 years ago

Closed 13 years ago

Last modified 11 years ago

#6486 closed bug (duplicate)

Cloning does not maintain checked state of checkboxes in IE8 standards mode.

Reported by: keith bentrup Owned by:
Priority: undecided Milestone:
Component: manipulation Version: 1.4.2
Keywords: ie8 standards mode checkbox Cc:
Blocked by: Blocking:

Description

In IE8 standards mode, if a user checks a checkbox and then that item is cloned, the checked state is not preserved. The problem is with the outerHTML value returned by the element and the method of cloning. A check for the documentMode can be used to correct the behavior.

Attachments (1)

ie8-test.html (1.0 KB) - added by keith bentrup 13 years ago.
cloning checkbox behavior test

Download all attachments as: .zip

Change History (7)

Changed 13 years ago by keith bentrup

Attachment: ie8-test.html added

cloning checkbox behavior test

comment:1 Changed 13 years ago by keith bentrup

A useful test in the clone function of manipulation.js might be:

if (document.documentMode && this.checked) {

some code

}

comment:2 Changed 13 years ago by keith bentrup

sorry that should be if (document.documentMode == 8 && this.checked)

comment:3 Changed 13 years ago by keith bentrup

http://github.com/jquery/jquery/blob/master/src/manipulation.js#L204

// Handle the case in IE 8 where action=/test/> self-closes a tag
.replace(/\=([^="'>\s]+\/)>/g, '="$1">')
.replace(rleadingWhitespace, "")], ownerDocument)[0];

changed to

// Handle the case in IE 8 where action=/test/> self-closes a tag
.replace(/=([^="'>\s]+\/)>/g, '="$1">')
// Handle IE 8 case where initially unchecked checkboxes and radio buttons (now checked) do not clone properly
.replace(/(<input)/i,(document.documentMode == 8 && this.checked ? '$1 checked="checked"' : '$1'))
.replace(rleadingWhitespace, "")], ownerDocument)[0];

comment:4 Changed 13 years ago by snover

Milestone: 1.4.3

Resetting milestone to future.

comment:5 Changed 13 years ago by snover

Priority: undecided
Resolution: duplicate
Status: newclosed

comment:6 Changed 13 years ago by snover

Duplicate of #3879.

Note: See TracTickets for help on using tickets.