Opened 10 years ago

Last modified 6 years ago

#1415 confirmed Bug

empty div tags are removed

Reported by: alfonsoml Owned by:
Priority: Normal Milestone:
Component: Core : Parser Version: 3.0
Keywords: Discussion Cc:


switch to source and paste

<div id="black_box"></div> 

switch twice and you'll see that the content has been lost.

(any browser)

Reported in

Change History (3)

comment:1 Changed 10 years ago by fredck

  • Keywords Discussion added
  • Priority changed from High to Normal

It doesn't happen if we have more content, like:

<div id="black_box"></div>

It is much probably is related to the cleanup we do to avoid having <p></p> or <div></div> tags, retrieving empty content to indicate that the editor has been left empty.

Suppose a user starts typing inside a <div>, and then delete everything with backspace. People claimed to have the editor posting empty, instead of <div></div>. But the ticket comes with another user case, where the editor is used to place an empty container.

What is the desired behavior here, to be able to handle all user cases?

comment:2 Changed 10 years ago by alfonsoml

Some quick ideas:

  • always, if the editor is in source mode, no clean up should be done (I think that it's the current behavior)
  • If the content is the same that the initial text, then assume that it is right and it shouldn't be cleaned
  • probably a single empty <p> or <br> (with nothing else) could always be cleaned
  • if the content is a div and enter mode!= div then don't clean it
  • if it has some attribute like class or id, then it probably shouldn't be cleaned

comment:3 Changed 6 years ago by krst

  • Component changed from General to Core : Parser
  • Status changed from new to confirmed
  • Version changed from FCKeditor 2.4.3 to 3.0

Actual also for CKE3.6.2Trunk

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