Opened 14 years ago

Closed 14 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 14 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 14 years ago by Alfonso Martínez de Lizarrondo

Milestone: CKEditor 3.3
Priority: HighNormal

comment:3 Changed 14 years ago by Michael Bu

Milestone: CKEditor 3.3

I think this defect should be fixed ASAP.

comment:4 Changed 14 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 14 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 14 years ago by Michael Bu

Milestone: CKEditor 3.4

comment:7 Changed 14 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 14 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 14 years ago by Michael Bu

Hope this could be fixed soon...

comment:10 Changed 14 years ago by Garry Yao

Resolution: expired
Status: newclosed

Already get fixed by #5310.

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy