ckeditor4 Two images get pasted in Safari when you paste a single image from MS word into the editor on the demo page

Type of report

Bug

Provide detailed reproduction steps (if any)

Go to CKEditor demo page
Copy an image from word file (e.g. img.docx)
Paste into the editor.
Two images get pasted in the editor instead of just one.

Expected result

What is the expected result of the above steps? Only single image gets pasted in the editor.

Actual result

What is the actual result of the above steps? 2 same images get pasted in the editor. The issue is that evt.data,dataTransfer.getFilesCount() returns 2 instead of 1.

Other details

Browser: Safari
OS: Mac
CKEditor version: 4.13.0
Installed CKEditor plugins: default plugins on demo page

Dec 12 '19 23:12 zoarif

Hello, thank you for filing this issue. Unfortunately I wasn't able to reproduce it. Could you provide more details? I need to know if you copy an image from MS Word Application or different one? If it's Word, which version? Also please attach exemplary document from which you are copying.

Dec 13 '19 10:12 Dumluregn

img.docx Yes it should be reproducible with word document. Make sure you try it on Mac(Safari)

Dec 13 '19 19:12 zoarif

Thank you! I reproduced this issue. Please notice it only happens on demo page - pasting to any of our CDN distributions (e.g. newest 4.13.1) doesn't paste image at all (which is caused by Safari-specific security measures - you can still paste image with any other content). As it seems to be just a demo page issue, allow me to change the issue title (and it will probably be referenced in another repo).

Dec 16 '19 10:12 Dumluregn

I also attached your example file to the issue description.

Dec 16 '19 10:12 Dumluregn

What is the status of this bug? Are you planning on fixing this?

Jul 09 '20 19:07 zoarif

Hello @zoarif, we have no ETA for this issue at the moment due to other priorities.

Jul 10 '20 06:07 f1ames

I've experienced something similar, but on Firefox 85.0.2 . Will this get fixed any time soon?

Feb 13 '21 19:02 whateverusernameisnottaken

Hello @whateverusernameisnottaken, does it occur on our demo page or in any other editor instance (like CDN 4.16.0) too?

Feb 15 '21 08:02 Dumluregn

It's not happening on the demo page, but it doesn't always happen on the wiki on which I'm working (FreshTomato router firmware documentation).

On the other link you posted, it won't even paste at all in there. ??

Feb 15 '21 17:02 whateverusernameisnottaken

D'OH! I have been usking CKGEdit. Isn't that a different editor entirely?

Feb 15 '21 18:02 whateverusernameisnottaken

It is based on CKEditor, however there may be substantial differences in how they work, so you should probably report a problem in CKGEdit repo as we are not the owners of that code.

Feb 15 '21 18:02 Dumluregn

Nope, I was right the first time. It was ckedit actually. Whew.

Feb 16 '21 03:02 whateverusernameisnottaken

dumluregn:

It's definitely ckedit I'm using. You're saying I shouldn't consider this a ckgedit bug and report it there?

Feb 19 '21 15:02 whateverusernameisnottaken

To be honest I'm lost which editor you are using. If it's CKGEdit then yes, you should report an issue elsewhere. I'm unfamiliar with ckedit, so I don't know if you meant CKEditor 4 by this or some other editor.

Feb 19 '21 15:02 Dumluregn

Sorry, I was doing doing too many things at once. I just looked and it is ckeditor 4.11.4 . That is this project, I believe.

Feb 19 '21 17:02 whateverusernameisnottaken

It is, though version 4.11.4 is over 2 years old and we introduced a lot of pasting improvements since. Please verify if you can reproduce the problem with the latest version (4.16.0).

Feb 22 '21 09:02 Dumluregn