Opened 7 years ago

Closed 6 years ago

#5381 closed Bug (fixed)

Cursor in wrong position after pasting content

Reported by: wwalc Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.3
Component: General Version: 3.1
Keywords: Confirmed Review+ Cc:

Description (last modified by wwalc)

Confirmed in IE8. Looks like a IE issue and it looks like it didn't occur before CKEditor 3.1 and probably it has something to do with pasting mixed <div> and <p> tags.

After pasting certain content, cursor is not positioned at the end of text but somewhere in the middle.

Steps to reproduce

  • Open in IE8 http://ckeditor.com/demo and select all text in the CKEditor and clear the contents.
  • Paste the content from example2.html - the cursor will be positioned at the beginning of the second line

(Old steps to reproduce below, the example above is much simpler)

  • Press "New Page" button and paste the content from example1.html - the cursor will be positioned at the beginning of the last paragraph - before "The content of... "
  • Press "New Page" button and paste the content from ckeditor_ie8_pasting_bug.CLP (use clipbrd.exe / ClipBoardViewer to open the file) - the cursor will be positioned at the end of first paragraph (after Gruss, schöne Test, Email)

Attachments (4)

ckeditor_ie8_pasting_bug.CLP (999 bytes) - added by wwalc 7 years ago.
example1.htm (5.1 KB) - added by wwalc 7 years ago.
example2.htm (51 bytes) - added by wwalc 7 years ago.
5381.patch (1.7 KB) - added by garry.yao 7 years ago.

Download all attachments as: .zip

Change History (9)

Changed 7 years ago by wwalc

Changed 7 years ago by wwalc

comment:1 Changed 7 years ago by wwalc

  • Description modified (diff)

Changed 7 years ago by wwalc

Changed 7 years ago by garry.yao

comment:2 Changed 7 years ago by garry.yao

  • Milestone set to CKEditor 3.3
  • Owner set to garry.yao
  • Status changed from new to assigned

It's an important fix, we should target it in 3.3.

comment:3 Changed 7 years ago by garry.yao

  • Keywords Review? added

comment:4 Changed 6 years ago by fredck

  • Keywords Review+ added; Review? removed

As the patch is a bit old, please keep attention to not duplicate the isNotWhitespace variable declaration when applying it (it happened to me).

comment:5 Changed 6 years ago by garry.yao

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

Fixed with [5462].

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