Opened 8 years ago

Closed 7 years ago

#4898 closed Bug (fixed)

Paste: After pasting a table from word it is not possible to navigate outside the table

Reported by: Damian Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.3
Component: General Version: SVN (CKEditor) - OLD
Keywords: Confirmed IE IBM Review+ Cc:

Description

After inserting a table using the paste function, it is not possible to navigate out of the table and continue editing the document.

Reproduced in FF.

Attachments (2)

test.doc (24.0 KB) - added by Damian 8 years ago.
4898.patch (2.5 KB) - added by Garry Yao 7 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 8 years ago by Garry Yao

Component: GeneralCore : Pasting
Keywords: Pending added
Milestone: CKEditor 3.1

WFM, could you attach a sample document for reproducing?

Changed 8 years ago by Damian

Attachment: test.doc added

comment:2 Changed 8 years ago by Damian

Added a test doc.

To reproduce select only the table without selecting any content outside the table and copy + paste.

Reproduced in IE 7. Seems to be OK in FF 3.

comment:3 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Confirmed IE added; Pending removed
Milestone: CKEditor 3.3

Hum... I'm a bit confused. The original report says it's confirmed with FF. Now, it's ok with FF, but buggy with IE.

Anyway... confirmed with IE8. The padding paragraph never gets created, so we can't move after the table.

comment:4 Changed 7 years ago by Garry Yao

Version: SVN (CKEditor)

Similar with #3336, the table was result inside the existing paragraph, the cure is to have special treatment in 'insertHtml'.

comment:5 Changed 7 years ago by Garry Yao

Milestone: CKEditor 3.3CKEditor 3.4

Changed 7 years ago by Garry Yao

Attachment: 4898.patch added

comment:6 Changed 7 years ago by Garry Yao

Component: Core : PastingGeneral
Keywords: Review? added
Milestone: CKEditor 3.4CKEditor 3.3
Owner: set to Garry Yao
Status: newassigned

Before correcting the 'insertHtml' behavior, we could still find a fix in our padding body logics to deal with such special case.

comment:7 Changed 7 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:8 Changed 7 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [5284].

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