Opened 3 years ago

Closed 3 years ago

#11839 closed Bug (fixed)

[IE9] Text cursor jumps out of source area on resize

Reported by: jozsefilles Owned by: Reinmar
Priority: Normal Milestone: CKEditor 4.4.1
Component: UI : Source View Version: 4.0
Keywords: IE9 Cc: byran.zaugg@…

Description

We've found the text cursor to jump out of the source-mode textarea after resize. Then the user can enter text underneath the textbox. This affects Internet Explorer 9.

Reproduction steps:

  1. Navigate to http://ckeditor.com/demo (using Internet Explorer 9)
  2. Switch to Source mode
  3. Click into the textarea
  4. Drag the resize handle in the bottom right corner
  5. Text cursor jumps out of the textarea
  6. Type some characters
  7. New text will appear under the area with different style

I found it to be related to an IE-specific resize workaround in CKEditor. On resize, the textarea is hidden for a moment but focus isn't restored after that.

We've successfully fixed it in our in-house fork.

Attachments (2)

11839.patch (1.9 KB) - added by jozsefilles 3 years ago.
2014-04-23_1510.swf (885.4 KB) - added by j.swiderski 3 years ago.

Download all attachments as: .zip

Change History (7)

Changed 3 years ago by jozsefilles

comment:1 Changed 3 years ago by blzaugg

  • Cc byran.zaugg@… added
  • Component changed from General to UI : Source View
  • Summary changed from Text cursor jumps out of source area on resize (IE9) to [IE9] Text cursor jumps out of source area on resize
  • Version set to 4.3.4

comment:2 Changed 3 years ago by j.swiderski

  • Keywords IE9 added
  • Status changed from new to confirmed
  • Version changed from 4.3.4 to 4.0

Problem can be reproduced from CKEditor 4.0 (works fine in beta) in IE9 only.

Changed 3 years ago by j.swiderski

comment:3 Changed 3 years ago by Reinmar

  • Milestone set to CKEditor 4.4.1

Thanks! Scheduled for review. If the provided patch is ok, we'll include it in 4.4.1.

comment:4 Changed 3 years ago by Reinmar

  • Owner set to Reinmar
  • Status changed from confirmed to assigned

comment:5 Changed 3 years ago by Reinmar

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

Merged to master with git:0e78956.

Thanks for the patch! It works, although I targeted the fix only for IE9. Other IEs blur editable and other browsers keep focus in editable when resizing, so I didn't want to change that behaviour (which is better in both cases). I also checked if the hack affects performance, but I didn't notice any change.

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