Opened 8 years ago

Closed 8 years ago

Last modified 8 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 8 years ago.
4944.patch (1.5 KB) - added by Garry Yao 8 years ago.
4944_2.patch (4.5 KB) - added by Garry Yao 8 years ago.
4944.doc (33.5 KB) - added by Garry Yao 8 years ago.
MS-Word document for reproducing regression of 4944.patch

Download all attachments as: .zip

Change History (14)

Changed 8 years ago by Damian

Attachment: MultiLevelList.doc added

comment:1 Changed 8 years ago by Tobiasz Cudnik

Keywords: Confirmed added

Reproduced in FF3.5 win.

Changed 8 years ago by Garry Yao

Attachment: 4944.patch added

comment:2 Changed 8 years ago by Garry Yao

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

comment:3 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:4 Changed 8 years ago by Garry Yao

Keywords: Review- added; Review+ removed

The changes have brought a regression, see #4946.

Changed 8 years ago by Garry Yao

Attachment: 4944_2.patch added

comment:5 Changed 8 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 8 years ago by Garry Yao

Attachment: 4944.doc added

MS-Word document for reproducing regression of 4944.patch

comment:6 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:7 Changed 8 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [4872].

comment:8 Changed 8 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 8 years ago by Garry Yao

Resolution: fixed
Status: reopenedclosed

Post-fixed with [4893].

comment:10 Changed 8 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 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy