Ticket #6127 (review_failed Bug)

Opened 4 years ago

Last modified 4 years ago

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

Reported by: satya Owned by: tobiasz.cudnik
Priority: Normal Milestone:
Component: General Version: 3.2.2
Keywords: IBM Cc: damo,joek

Description (last modified by wwalc) (diff)

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

6127.patch (680 bytes) - added by tobiasz.cudnik 4 years ago.

Change History

comment:1 Changed 4 years ago by satya

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 4 years ago by tobiasz.cudnik

  • Owner set to tobiasz.cudnik
  • Status changed from new to assigned

comment:3 follow-up: ↓ 4 Changed 4 years ago by 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?

comment:4 in reply to: ↑ 3 ; follow-up: ↓ 6 Changed 4 years ago by satya

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 4 years ago by tobiasz.cudnik

comment:5 Changed 4 years ago by tobiasz.cudnik

  • Status changed from assigned to review

comment:6 in reply to: ↑ 4 Changed 4 years ago by fredck

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 4 years ago by garry.yao

  • Status changed from review to review_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 4 years ago by damo

Will this fix make it into 3.4.2?

comment:9 Changed 4 years ago by wwalc

  • Description modified (diff)
Note: See TracTickets for help on using tickets.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy