Opened 8 years ago
Last modified 8 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> </p> <ol><br /> <li> </li> </ol> <p> </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 8 years ago by
Status: | new → pending |
---|---|
Version: | 4.5.11 |
comment:2 Changed 8 years ago by
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 8 years ago by
Status: | pending → confirmed |
---|---|
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.
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?