#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 6 years ago by
| Component: | unfiled → manipulation |
|---|---|
| Milestone: | 1.next → 1.5.1 |
| Priority: | undecided → high |
| Status: | new → open |
comment:2 Changed 6 years ago by
comment:3 Changed 6 years ago by
| Priority: | high → blocker |
|---|
comment:4 Changed 6 years ago by
| Keywords: | regression added |
|---|
comment:6 Changed 6 years ago by
| Owner: | set to Rick Waldron |
|---|---|
| Status: | open → assigned |
comment:7 Changed 6 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → 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.