Opened 9 years ago
Closed 9 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
- Open Nightly build in IE & add 2 paragraphs of text
- Select all content using CTRL + A & copy it using CTRL + C
- Open same nightly build in Chrome
- 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 9 years ago by
Milestone: | → CKEditor 4.5.4 |
---|
comment:2 Changed 9 years ago by
Status: | new → confirmed |
---|---|
Version: | 4.5.3 → 4.5.2 |
I have been able to reproduce this issue from CKEditor 4.5.2 in IE/Chrome.
comment:3 Changed 9 years ago by
Owner: | set to Tade0 |
---|---|
Status: | confirmed → assigned |
comment:4 Changed 9 years ago by
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 9 years ago by
Status: | assigned → review |
---|
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 9 years ago by
Resolution: | → fixed |
---|---|
Status: | review → closed |
Superb! Fixed on master with git:d232212.
I haven't yet checked it, but it sounds reasonably and hence I assigned it to the milestone.