Major defect with enterMode = ENTER_BR

[Michael Bu]

For a HTML code like this:

some_text_here<img src="xxx"/>

With enterMode = ENTER_BR, hitting "Enter" between the text and img would show the line break being inserted in WYSIWYG mode correctly, but switching to SOURCE mode you'll see the BR tag was added after the IMG tag.

As a result, the actual code being submitted to server side was incorrect. This is a major defect as users thought they had the text formatted correctly in the editor but shown up otherwise.

[Michael Bu]

Another scenario is for two consecutive IMG tags like this:

<img src="xxx"/><img src="yyy"/>

With enterMode=ENTER_BR, inserting line breaks between two images would be almost impossible...

[Michael Bu]

I think this defect should be fixed ASAP.

[Alfonso Martínez de Lizarrondo]

Provide a patch for it if you are in such a hurry. There are other bugs that also have to be fixed.

[Michael Bu]

Too bad I have no idea how to get this fixed. Seem to be a core related issue.

Shouldn't you at least set a milestone for it? Or this bug would probably never gets fixed...

[Alfonso Martínez de Lizarrondo]

Please, don't set any milestone. That's the best way to make us ignore a bug.

[Wiktor Walc]

Confirmed in IE8. Looks like a regression bug introduced in 3.1.1.

Unfortunately this bug exists also when enterMode is set to default value, it can be reproduced on the CKEditor demo web site.

Due to this bug CKEditor is destroying the content.

Steps to reproduce

• Open ​http://ckeditor.com/demo in IE8
• Switch to source mode, clear the contents and paste the following:

  <p>
  	aaaaaa
  <p style="width:130px">
  	11 
  	<br />
  	<br />
  	<br />
  	<br />
  	<br />
  	<br />
  	<br />
  <img alt="" src="http://a.cksource.com/c/1/inc/img/demo-little-red.jpg" style="width: 120px; float: right; height: 168px; margin-left: 10px; margin-right: 10px" /><br />
  	22</p>
  <p>
  	bbbbb</p>
  

• switch to wysiwyg mode and back to source mode
• result:

  <p>
  	aaaaaa</p>
  <p style="width: 130px">
  	11 <img alt="" src="http://a.cksource.com/c/1/inc/img/demo-little-red.jpg" style="width: 120px; float: right; height: 168px; margin-left: 10px; margin-right: 10px" /><br />
  	<br />
  	<br />
  	<br />
  	<br />
  	<br />
  	<br />
  	<br />
  	22</p>
  <p>
  	bbbbb</p>
  

[Michael Bu]

Hope this could be fixed soon...

[Garry Yao]

Already get fixed by #5310.