Opened 14 years ago

Closed 14 years ago

#5381 closed Bug (fixed)

Cursor in wrong position after pasting content

Reported by: Wiktor Walc Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.3
Component: General Version: 3.1
Keywords: Confirmed Review+ Cc:

Description (last modified by Wiktor Walc)

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 Wiktor Walc 14 years ago.
example1.htm (5.1 KB) - added by Wiktor Walc 14 years ago.
example2.htm (51 bytes) - added by Wiktor Walc 14 years ago.
5381.patch (1.7 KB) - added by Garry Yao 14 years ago.

Download all attachments as: .zip

Change History (9)

Changed 14 years ago by Wiktor Walc

Changed 14 years ago by Wiktor Walc

Attachment: example1.htm added

comment:1 Changed 14 years ago by Wiktor Walc

Description: modified (diff)

Changed 14 years ago by Wiktor Walc

Attachment: example2.htm added

Changed 14 years ago by Garry Yao

Attachment: 5381.patch added

comment:2 Changed 14 years ago by Garry Yao

Milestone: CKEditor 3.3
Owner: set to Garry Yao
Status: newassigned

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

comment:3 Changed 14 years ago by Garry Yao

Keywords: Review? added

comment:4 Changed 14 years ago by Frederico Caldeira Knabben

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 14 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [5462].

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy