‘ignoreEmptyParagraph’ configuration doesn't work in enterMode=BR

[Garry Yao]

Environment

Firefox, config.enterMode = CKEDITOR.ENTER_BR, config.ignoreEmptyParagraph = true;

Reproducing Procedures

1. Open any of the sample page and click on 'New Page' to clear content.
2. Swtich to source.
   • Actual Result: There's one single <br /> left in content .

[giorez]

I tested your patch, but no luck to totally solve the issue. Pls see ​http://drupal.org/node/798932#comment-2979676. I'm using CKEditor 3.2.1.

[Alfonso Martínez de Lizarrondo]

I don't notice any problem about "text getting bumped down upon initial typing", but if the user inserts several empty lines at the end of the content, they go away one at a time with each switch between source & design, so we have a new problem.

As the bogus node is marked as "_moz_editor_bogus_node" maybe that can be used to filter it out in the node: in the getData method of the wysiwygarea plugin, after getting the initial content, test (only for gecko) if the last node of the DOM is a br and has this attribute and if it's true then replace it from the end of the document.

How does that sound?

[Garry Yao]

they go away one at a time with each switch between source & design, so we have a new problem.

Unfortunately it's a bug that exists now, I've opened #5766 for it.

[Garry Yao]

Replying to alfonsoml:

As the bogus node is marked as "_moz_editor_bogus_node" maybe that can be used to filter it out in the node: in the getData method of the wysiwygarea plugin, after getting the initial content, test (only for gecko) if the last node of the DOM is a br and has this attribute and if it's true then replace it from the end of the document.

Firfox bogus nodes are imitated into normal BR, it's safe to just remove the last <br> of document.

But the problem itself is off the scope on this ticket, it should be instead discussed at #5293.

[Frederico Caldeira Knabben]

This is definitely supposed to be handled by #5293.

[Frederico Caldeira Knabben]

Confirmed it fixed at #5293.