#4944 closed Bug (fixed)
Paste: Problem with multi-level lists
| Reported by: | Damian | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.1 |
| Component: | Core : Pasting | Version: | SVN (CKEditor) - OLD |
| Keywords: | IBM Confirmed Review+ | Cc: | Joe Kavanagh |
Description
Some multi level bullet lists do not paste correctly.
Please refer to test document attached.
Reproduced in IE 6 & 7 and FF 2 & 3
Attachments (4)
Change History (14)
Changed 8 years ago by
| Attachment: | MultiLevelList.doc added |
|---|
comment:1 Changed 8 years ago by
| Keywords: | Confirmed added |
|---|
Changed 8 years ago by
| Attachment: | 4944.patch added |
|---|
comment:2 Changed 8 years ago by
| Keywords: | Review? added |
|---|---|
| Owner: | set to Garry Yao |
| Status: | new → assigned |
| Version: | → SVN (CKEditor) |
comment:3 Changed 8 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
comment:4 Changed 8 years ago by
| Keywords: | Review- added; Review+ removed |
|---|
The changes have brought a regression, see #4946.
Changed 8 years ago by
| Attachment: | 4944_2.patch added |
|---|
comment:5 Changed 8 years ago by
| Keywords: | Review? added; Review- removed |
|---|
Proposing a innovative way of figuring out 'indent unit' by looking at the margin increament instead of the first margin value, which is more reliable (in case that the first list item also has an additional indentation).
Changed 8 years ago by
MS-Word document for reproducing regression of 4944.patch
comment:6 Changed 8 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
comment:8 Changed 8 years ago by
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
The original issue can still be reproduced when using the attached MultiLevelList.doc test case. Notice that the first list in the test case does not work and the second one does. The difference between the lists is that there are no level 2 items beneath the level 3 items.
comment:9 Changed 8 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
Post-fixed with [4893].
comment:10 Changed 8 years ago by
Unfortunately this comes too later for 3.1, we'll target it to the next maintaining release.
Reproduced in FF3.5 win.