Opened 9 years ago

Closed 9 years ago

#5337 closed Bug (expired)

Major defect with enterMode = ENTER_BR

Reported by: Michael Bu Owned by:
Priority: Normal Milestone:
Component: UI : Enter Key Version: 3.1.1
Keywords: Confirmed Cc:

Description

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.

Change History (10)

comment:1 Changed 9 years ago by 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...

comment:2 Changed 9 years ago by Alfonso Martínez de Lizarrondo

Milestone: CKEditor 3.3
Priority: HighNormal

comment:3 Changed 9 years ago by Michael Bu

Milestone: CKEditor 3.3

I think this defect should be fixed ASAP.

comment:4 Changed 9 years ago by Alfonso Martínez de Lizarrondo

Milestone: CKEditor 3.3

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

comment:5 Changed 9 years ago by 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...

comment:6 Changed 9 years ago by Michael Bu

Milestone: CKEditor 3.4

comment:7 Changed 9 years ago by Alfonso Martínez de Lizarrondo

Milestone: CKEditor 3.4

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

comment:8 Changed 9 years ago by Wiktor Walc

Keywords: Confirmed added
Version: 3.23.1.1

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>
    

comment:9 Changed 9 years ago by Michael Bu

Hope this could be fixed soon...

comment:10 Changed 9 years ago by Garry Yao

Resolution: expired
Status: newclosed

Already get fixed by #5310.

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