Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#12796 closed Bug (fixed)

Indent List plugin unwraps parent li elements

Reported by: astucki Owned by:
Priority: Normal Milestone: CKEditor 4.4.8
Component: Core : Lists Version: 4.0 Beta
Keywords: Cc: andrew.stucki@…

Description

The example setup is here:

​http://jsfiddle.net/hruygj01/2/

Highlight both the ordered list and unordered list in either inline editor and hit the button to decrease indentation level. Rather than unwrapping the first list like what happens when the parent of the editor is a div, the parent itself is unwrapped.

Attachments (1)

inlineall2.html​ (1.5 KB) - added by Jakub Ś 12 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 12 years ago by astucki

Cc: andrew.stucki@… added
Component: General → Core : Lists
Version: → 4.4.6

comment:2 Changed 12 years ago by Jakub Ś

Status: new → confirmed
Version: 4.4.6 → 4.0 Beta

Problem can be reproduced in every browser from CKEditor 4.0 beta.

The condition is that you need to select whole list. Indenting single list entries works as expected.

Changed 12 years ago by Jakub Ś

Attachment: inlineall2.html​ added

comment:3 Changed 12 years ago by astucki

Linking ​https://github.com/ckeditor/ckeditor-dev/pull/155 for possible/example fix.

comment:4 Changed 12 years ago by Piotr Jasiun

Resolution: → fixed
Status: confirmed → closed

Closed thanks to ​andrewstucki.

Discussion on GitHub.

​git:6a8dc27

comment:5 Changed 12 years ago by Piotr Jasiun

Milestone: → CKEditor 4.4.8
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