Opened 14 years ago

Last modified 14 years ago

#6127 review_failed Bug

IE Cursor not going to next line when we press enter & we have set forceEntermode=br as config option

Reported by: Satya Minnekanti Owned by: Tobiasz Cudnik
Priority: Normal Milestone:
Component: General Version: 3.2.2
Keywords: IBM Cc: Damian, joek

Description (last modified by Wiktor Walc)

To reproduce the defect:

  1. set enterMode = CKEDITOR.ENTER_BR in the config option.
  1. Open Ajax sample. Type some text.
  1. Press Enter at the end of the line.

Expected Result:

New line is created and cursor is shown in the new line.

Actual Result:

New line is created but Cursor is shown in the same line from where we have pressed Enter.

Cursor will move to Correct line only when user starts typing the text.

Tested in IE 6&7

Attachments (1)

6127.patch (680 bytes) - added by Tobiasz Cudnik 14 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 14 years ago by Satya Minnekanti

This is a correction for step 3..

select all the text that you have typed in step 2 & apply background color to the text, go to end of the line and then press Enter..

comment:2 Changed 14 years ago by Tobiasz Cudnik

Owner: set to Tobiasz Cudnik
Status: newassigned

comment:3 Changed 14 years ago by Frederico Caldeira Knabben

@satya, just to be sure... forceEntermode is a boolean, so it can't be set to br, only to true/false. Aren't you talking about the "enterMode" setting instead?

comment:4 in reply to:  3 ; Changed 14 years ago by Satya Minnekanti

Replying to fredck:

@satya, just to be sure... forceEntermode is a boolean, so it can't be set to br, only to true/false. Aren't you talking about the "enterMode" setting instead?

Hi fred, yes i am talking about "enterMode" it was mistakenly typed as forceEntermode

Changed 14 years ago by Tobiasz Cudnik

Attachment: 6127.patch added

comment:5 Changed 14 years ago by Tobiasz Cudnik

Status: assignedreview

comment:6 in reply to:  4 Changed 14 years ago by Frederico Caldeira Knabben

Replying to satya:

yes i am talking about "enterMode" it was mistakenly typed as forceEntermode

Satya, we *strongly* recommend NOT to use enterMode, leaving it's default setting. If you're using it in your applications, you should definitely consider changing it.

PS: This is just a note, which has nothing to do with the validity of this ticker.

comment:7 Changed 14 years ago by Garry Yao

Status: reviewreview_failed

It's not a good thing to introduce NBSP inside the editor without collecting it, the correct fix should instead make BR created outside of inline element.

<span>text</span><br />
INSTEAD OF
<span>text<br /></span>

comment:8 Changed 14 years ago by Damian

Will this fix make it into 3.4.2?

comment:9 Changed 14 years ago by Wiktor Walc

Description: modified (diff)
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