Ticket #7433 (closed Bug: fixed)

Opened 3 years ago

Last modified 7 months ago

[IE9] enterBr doesn't put cursor to next line

Reported by: garry.yao Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.5.3
Component: UI : Enter Key Version: 3.0
Keywords: IE9 Cc:

Description

With IE9 standard mode:

  1. Load any of the sample page;
  2. Put cursor at the end of block and press one shift enter;
  3. Press shift enter again;
    • Expected Result: Cursor moved to the next line;
    • Actual Result: Cursor stays at the same line.

Attachments

7433.patch (1.5 KB) - added by garry.yao 3 years ago.

Change History

comment:1 Changed 3 years ago by garry.yao

  • Status changed from new to confirmed
  • Milestone set to CKEditor 3.5.3

Changed 3 years ago by garry.yao

comment:2 Changed 3 years ago by garry.yao

  • Status changed from confirmed to review
  • Owner set to garry.yao

This's a new IE9 regression that report wrong range position for any range anchors besides BR node.

The workaround is a about a new (and shorter) way to determinate IE selection range, it create an empty anchor element as marker and report its position instead of relying on traditional position comparison approach.

comment:3 Changed 3 years ago by Saare

  • Status changed from review to review_passed

Let's file a IE bug.

comment:4 Changed 3 years ago by garry.yao

  • Status changed from review_passed to closed
  • Resolution set to fixed

IE bug filed.

Fixed with [6624].

comment:5 Changed 7 months ago by berend_engelbrecht

This problem came back in IE10. Steps to reproduce:

  • Download latest stable ckeditor
  • Add this line in config.js: config.enterMode = CKEDITOR.ENTER_BR;
  • Try any of the samples in IE10 standard mode.

Note that every second enter key press does not result in a new line in the editor.

The IE-specific code in changeset 6624 that fixes Ticket #7433 is only reached if .ie9Compat evaluates to true. Replace .ie9Compat by .ie && .version >= 9 and the problem is resolved in ie10 as well.

Please note that the change for ticket #9535 (with similar description) does not fix this problem.

comment:6 Changed 7 months ago by j.swiderski

@berend_engelbrecht I have reported #10943.

comment:7 Changed 7 months ago by Reinmar

I tried to reproduce this on IE9 and IE10 on 4.2.1 and latest master with no luck.

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