#8129 closed bug (fixed)
Cloning <select multiple> selected items incorrect in IE8
| Reported by: | aaronbarker | Owned by: | rwaldron |
|---|---|---|---|
| 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 6 years ago by jitter
- Component changed from unfiled to manipulation
- Milestone changed from 1.next to 1.5.1
- Priority changed from undecided to high
- Status changed from new to open
comment:2 Changed 6 years ago by dmethvin
comment:3 Changed 6 years ago by jitter
- Priority changed from high to blocker
comment:4 Changed 6 years ago by jitter
- Keywords regression added
comment:5 Changed 6 years ago by rwaldron
comment:6 Changed 6 years ago by rwaldron
- Owner set to rwaldron
- Status changed from open to assigned
comment:7 Changed 6 years ago by rwldrn
- Resolution set to fixed
- Status changed from assigned to closed
Fixes #8129. Fix cloning multiple selected options in IE8.
Changeset: fa4c90987fc9aede4de3f41976cda91acad81736
Note: See
TracTickets for help on using
tickets.
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.