Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#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)

MultiLevelList.doc (20.0 KB) - added by Damian 14 years ago.
4944.patch (1.5 KB) - added by Garry Yao 14 years ago.
4944_2.patch (4.5 KB) - added by Garry Yao 14 years ago.
4944.doc (33.5 KB) - added by Garry Yao 14 years ago.
MS-Word document for reproducing regression of 4944.patch

Download all attachments as: .zip

Change History (14)

Changed 14 years ago by Damian

Attachment: MultiLevelList.doc added

comment:1 Changed 14 years ago by Tobiasz Cudnik

Keywords: Confirmed added

Reproduced in FF3.5 win.

Changed 14 years ago by Garry Yao

Attachment: 4944.patch added

comment:2 Changed 14 years ago by Garry Yao

Keywords: Review? added
Owner: set to Garry Yao
Status: newassigned
Version: SVN (CKEditor)

comment:3 Changed 14 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:4 Changed 14 years ago by Garry Yao

Keywords: Review- added; Review+ removed

The changes have brought a regression, see #4946.

Changed 14 years ago by Garry Yao

Attachment: 4944_2.patch added

comment:5 Changed 14 years ago by Garry Yao

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 14 years ago by Garry Yao

Attachment: 4944.doc added

MS-Word document for reproducing regression of 4944.patch

comment:6 Changed 14 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:7 Changed 14 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [4872].

comment:8 Changed 14 years ago by Damian

Resolution: fixed
Status: closedreopened

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 14 years ago by Garry Yao

Resolution: fixed
Status: reopenedclosed

Post-fixed with [4893].

comment:10 Changed 14 years ago by Garry Yao

Unfortunately this comes too later for 3.1, we'll target it to the next maintaining release.

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy