Opened 7 years ago

Last modified 7 years ago

#6281 confirmed Bug

List items not wrapped in UL/OL render incorrectly

Reported by: Jennifer Owned by:
Priority: Normal Milestone:
Component: Core : Lists Version: 3.3.2
Keywords: Cc: ladybug_3777@…, rirwin@…

Description (last modified by Krzysztof Studnik)

In the old fckeditor I could enter the following (incorrect) HTML code in source mode:

<li>item one</li>
<li>item two</li>

When I toggled / switched back to wysiwyg mode fckeditor would realize I was trying to make a list and automatically append the proper UL tag around my list items like so:

<ul>
   <li>item one</li>
   <li>item two</li>
</ul>

I've noticed that ckeditor 3.3.2 does not automatically detect that I'm trying to create a list and not only does it not add in the ul tags but instead creates new lines (more and more of them) every time I toggle from source to wysiwyg like so: First toggle:

<p>&nbsp;</p>
<p>
<li>
	item one</li>
</p>
<p>&nbsp;</p>
<p>&nbsp;</p>
<p>
<li>
	item two</li>
</p>

Second toggle:

<p>&nbsp;</p>
<p>&nbsp;</p>
<p>&nbsp;</p>
<p>
<li>
	item one</li>
</p>
<p>&nbsp;</p>
<p>&nbsp;</p>
<p>&nbsp;</p>
<p>&nbsp;</p>
<p>&nbsp;</p>
<p>&nbsp;</p>
<p>
<li>
	item two</li>
</p>
<p>&nbsp;</p>

Making the problem worse each time.

The problem is more pronounced if I forget to put in wrapping ul/ol AND also forget closing li tags. So for example if I put this into source:

<li>item one
<li>item two

When I toggle out and back into source I now lose my first item completely and see this in source:

Code: Select all

<p>&nbsp;</p>
<p>
<li>
   two</li>
</p>

I realize users shouldn't be putting in bad list HTML code, but it seems unfortunate that the old version of the editor was able to handle this situation and the new editor makes a mess of it

I've posted about this issue in the forums here: http://cksource.com/forums/viewtopic.php?f=11&t=20104

Change History (3)

comment:1 Changed 7 years ago by Jennifer

Cc: ladybug_3777@… added

comment:2 Changed 7 years ago by Krzysztof Studnik

Description: modified (diff)
Keywords: UL LI OL List lists removed
Status: newconfirmed

Confirmed in 3.4 and 3.4.1

comment:3 Changed 7 years ago by Richard Irwin

Cc: rirwin@… added
Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy