Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#4944 closed Bug (fixed)

Paste: Problem with multi-level lists

Reported by: damo Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.1
Component: Core : Pasting Version: SVN (CKEditor) - OLD
Keywords: IBM Confirmed Review+ Cc: JoeK

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 damo 6 years ago.
4944.patch (1.5 KB) - added by garry.yao 6 years ago.
4944_2.patch (4.5 KB) - added by garry.yao 6 years ago.
4944.doc (33.5 KB) - added by garry.yao 6 years ago.
MS-Word document for reproducing regression of 4944.patch

Download all attachments as: .zip

Change History (14)

Changed 6 years ago by damo

comment:1 Changed 6 years ago by tobiasz.cudnik

  • Keywords Confirmed added

Reproduced in FF3.5 win.

Changed 6 years ago by garry.yao

comment:2 Changed 6 years ago by garry.yao

  • Keywords Review? added
  • Owner set to garry.yao
  • Status changed from new to assigned
  • Version set to SVN (CKEditor)

comment:3 Changed 6 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:4 Changed 6 years ago by garry.yao

  • Keywords Review- added; Review+ removed

The changes have brought a regression, see #4946.

Changed 6 years ago by garry.yao

comment:5 Changed 6 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 6 years ago by garry.yao

MS-Word document for reproducing regression of 4944.patch

comment:6 Changed 6 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:7 Changed 6 years ago by garry.yao

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed with [4872].

comment:8 Changed 6 years ago by damo

  • Resolution fixed deleted
  • Status changed from closed to 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 6 years ago by garry.yao

  • Resolution set to fixed
  • Status changed from reopened to closed

Post-fixed with [4893].

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