Ticket #4944 (closed Bug: fixed)

Opened 5 years ago

Last modified 5 years ago

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

MultiLevelList.doc (20.0 KB) - added by damo 5 years ago.
4944.patch (1.5 KB) - added by garry.yao 5 years ago.
4944_2.patch (4.5 KB) - added by garry.yao 5 years ago.
4944.doc (33.5 KB) - added by garry.yao 5 years ago.
MS-Word document for reproducing regression of 4944.patch

Change History

Changed 5 years ago by damo

comment:1 Changed 5 years ago by tobiasz.cudnik

  • Keywords Confirmed added

Reproduced in FF3.5 win.

Changed 5 years ago by garry.yao

comment:2 Changed 5 years ago by garry.yao

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

comment:3 Changed 5 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:4 Changed 5 years ago by garry.yao

  • Keywords Review- added; Review+ removed

The changes have brought a regression, see #4946.

Changed 5 years ago by garry.yao

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

MS-Word document for reproducing regression of 4944.patch

comment:6 Changed 5 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:7 Changed 5 years ago by garry.yao

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

Fixed with [4872].

comment:8 Changed 5 years ago by damo

  • Status changed from closed to reopened
  • Resolution fixed deleted

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 5 years ago by garry.yao

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

Post-fixed with [4893].

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