Opened 3 years ago

Closed 3 years ago

#13690 closed Bug (fixed)

Copying content from IE to Chrome adding extra paragraph

Reported by: Satya Minnekanti Owned by: Tade0
Priority: Normal Milestone: CKEditor 4.5.4
Component: General Version: 4.5.2
Keywords: IBM Cc: Irina

Description

Steps to reproduce

  1. Open Nightly build in IE & add 2 paragraphs of text
  2. Select all content using CTRL + A & copy it using CTRL + C
  3. Open same nightly build in Chrome
  4. Paste the content copied from IE browser in Step 2 in to Chrome

Expected result

2 Paragrphs pastd properly

Actual result

A new paragraph added above pasted content and it has following code

<p>Rich Text Editor, editor, Press ALT 0 for help<!--StartFragment--></p>

This is regression in 4.5. it's working properly in 4.4.8

==

Change History (6)

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

Milestone: CKEditor 4.5.4

I haven't yet checked it, but it sounds reasonably and hence I assigned it to the milestone.

comment:2 Changed 3 years ago by Jakub Ś

Status: newconfirmed
Version: 4.5.34.5.2

I have been able to reproduce this issue from CKEditor 4.5.2 in IE/Chrome.

comment:3 Changed 3 years ago by Tade0

Owner: set to Tade0
Status: confirmedassigned

comment:4 Changed 3 years ago by Tade0

This additional paragraph is the <title> within the source iframe.

Apparently after this commit: https://github.com/ckeditor/ckeditor-dev/commit/ca0851c7a14f616a0c4cda905816aa87ca399efb The content of the iframe is copied and pasted whole.

Working on a solution now.

comment:5 Changed 3 years ago by Tade0

Status: assignedreview

In the commit mentioned the content was pasted whole and filtered, but the filters were case-sensitive, so pasting HTML content with uppercase tags (as it is the case in IE) resulted in errors.

I made the filters case-insensitive. Changes pushed to branch:t/13690.

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

Resolution: fixed
Status: reviewclosed

Superb! Fixed on master with git:d232212.

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