Opened 10 years ago

Closed 10 years ago

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

Download all attachments as: .zip

Change History (14)

Changed 10 years ago by Damian

Attachment: MultiLevelList.doc added

comment:1 Changed 10 years ago by Tobiasz Cudnik

Keywords: Confirmed added

Reproduced in FF3.5 win.

Changed 10 years ago by Garry Yao

Attachment: 4944.patch added

comment:2 Changed 10 years ago by Garry Yao

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

comment:3 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:4 Changed 10 years ago by Garry Yao

Keywords: Review- added; Review+ removed

The changes have brought a regression, see #4946.

Changed 10 years ago by Garry Yao

Attachment: 4944_2.patch added

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

Attachment: 4944.doc added

MS-Word document for reproducing regression of 4944.patch

comment:6 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:7 Changed 10 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [4872].

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

Resolution: fixed
Status: reopenedclosed

Post-fixed with [4893].

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