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: Piotrek Koszuliński
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 Jakub Ś 3 years ago.

Download all attachments as: .zip

Change History (7)

Changed 3 years ago by jozsefilles

Attachment: 11839.patch added

comment:1 Changed 3 years ago by Byran Zaugg

Cc: byran.zaugg@… added
Component: GeneralUI : Source View
Summary: Text cursor jumps out of source area on resize (IE9)[IE9] Text cursor jumps out of source area on resize
Version: 4.3.4

comment:2 Changed 3 years ago by Jakub Ś

Keywords: IE9 added
Status: newconfirmed
Version: 4.3.44.0

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

Changed 3 years ago by Jakub Ś

Attachment: 2014-04-23_1510.swf added

comment:3 Changed 3 years ago by Piotrek Koszuliński

Milestone: 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 Piotrek Koszuliński

Owner: set to Piotrek Koszuliński
Status: confirmedassigned

comment:5 Changed 3 years ago by Piotrek Koszuliński

Resolution: fixed
Status: assignedclosed

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 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy