Opened 3 years ago

Last modified 3 years ago

#14866 confirmed Bug

Deleting bullet list does not work

Reported by: yev Owned by:
Priority: Normal Milestone:
Component: Core : Lists Version: 4.2
Keywords: Cc:

Description

Steps to reproduce

For the following markup, if you try to delete the bullet in wysiwyg (backspace), it will not get deleted.

<p>&nbsp;</p>

<ol><br />
	<li>&nbsp;</li>
</ol>

<p>&nbsp;</p>

if you toggle Source, the OL>LI will come back. Same thing will happen if you persist markup on the server and reload it into another instance of cke - you will get the bullet list back ever though you thought you had deleted it.

Expected result

A clean up of orphaned <OL> needs to occur

Actual result

<OL> remains when <LI> is removed

Other details (browser, OS, CKEditor version, installed plugins)

Chome 52.0.2743.116 (64-bit) Mac CKEditor 4.5.11

Change History (3)

comment:1 Changed 3 years ago by Jakub Ś

Status: newpending
Version: 4.5.11

According to http://www.w3.org/TR/html-markup/ul.html#ul or https://html.spec.whatwg.org/multipage/semantics.html#the-ul-element, permitted content inside ul are only li tags script-supporting elements so such content should not be used in the first place. What is more it is not possible to create such content in the editor. How are you creating such content?

Second thing is that I had no problem removing such list in Chrome on Win and Mac. I have pressed backspace 2 times and whole list was gone.

Are you able to reproduce this issue in demo page or in one of predefined packages for CKEditor installed as standalone application on server?

comment:2 Changed 3 years ago by yev

This markup was introduced manually via Source (I believe). I can reproduce it in demo page as well. Workarounds for a user exist if they know to look out for this problem. Backspace twice works, and via list toolbar also works. But single backspace may look like user deleted a list, yet they haven't.

comment:3 Changed 3 years ago by Jakub Ś

Status: pendingconfirmed
Version: 4.2

I have actually made a mistake when analysing this issue. You can remove the list with double backspace only when you paste just the list with br inside, into empty editor. If however you insert this list into existing content or with empty P above (just as you have shown in original TC) then it is not possible to remove this list anymore.

This problem can be reproduced from CKEditor 4.2.0


Please note that although this is an issue it is actually a marginal one. While it should be possible to remove this list (like in earlier versions), such list should not be entered into editor in a first place. This is invalid HTML and it is not possible to create such inside CKEditor. If someone is inserting HTML manually, he should know the thing or two about it.

I however agree it should defiitelly be possible to remove such list.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy