Bug Tracker

Opened 7 years ago

Closed 7 years ago

Last modified 5 years ago

#8129 closed bug (fixed)

Cloning <select multiple> selected items incorrect in IE8

Reported by: aaronbarker Owned by: Rick Waldron
Priority: blocker Milestone: 1.5.1
Component: manipulation Version: 1.5
Keywords: regression Cc:
Blocked by: Blocking:

Description

http://jsfiddle.net/aaronbarker/VSnzy/2/

The selected options are not cloned correctly. Options 2/4/6 should be selected, but in the clone only 6 is selected

This was working in 1.4.4 http://jsfiddle.net/aaronbarker/HJ3Wf/1/

Change History (7)

comment:1 Changed 7 years ago by jitter

Component: unfiledmanipulation
Milestone: 1.next1.5.1
Priority: undecidedhigh
Status: newopen

comment:2 Changed 7 years ago by dmethvin

It's due to the order that we call cloneFixAttributes. First we fix the attributes of all the descendants, then we fix the attrs of the parent. After the descendants are fixed everything looks good. However, when fixing the cloned parent it calls clearAttributes; that removes the multiple attribute and only leaves the last selected option standing.

comment:3 Changed 7 years ago by jitter

Priority: highblocker

comment:4 Changed 7 years ago by jitter

Keywords: regression added

comment:6 Changed 7 years ago by Rick Waldron

Owner: set to Rick Waldron
Status: openassigned

comment:7 Changed 7 years ago by rwldrn

Resolution: fixed
Status: assignedclosed

Fixes #8129. Fix cloning multiple selected options in IE8.

Changeset: fa4c90987fc9aede4de3f41976cda91acad81736

Note: See TracTickets for help on using tickets.