Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Browser freezes after dropping the image to the editor from the files browsing window #803

Closed
Mgsy opened this issue Feb 1, 2018 · 4 comments
Labels
browser:chrome type:bug This issue reports a buggy (incorrect) behavior.

Comments

@Mgsy
Copy link
Member

Mgsy commented Feb 1, 2018

🐞 Is this a bug report or feature request? (choose one)

  • Bug report

💻 Version of CKEditor

Latest master.

📋 Steps to reproduce

  1. Go to http://localhost:8125/ckeditor5-easy-image/tests/manual/easyimage.html.
  2. Click on the insert image icon.
  3. Drag the image to the editor.
  4. Close the window.

✅ Expected result

The image has been inserted and editor works fine.

❎ Actual result

The image has been inserted, but the browser freezes.

📃 Other details that might be useful

Notes

This bug is pretty random, I wasn't able to find any dependency while it occurred. Basically, I am able to reproduce this issue only in Chrome and on Windows. Sometimes it occurs after first d&d, sometimes after 10th and sometimes just doesn't occur.

Drag and drop upload from the desktop works fine, so it looks like problem occurs only when image is dragged from the files browsing window.

GIF

bug_cke5

Other information

OS: Windows 10
Browser: Chrome

@Mgsy Mgsy added the type:bug This issue reports a buggy (incorrect) behavior. label Feb 1, 2018
@oleq
Copy link
Member

oleq commented Feb 1, 2018

That's a nasty one. It means that users might lose their content.

cc @Reinmar

@Reinmar
Copy link
Member

Reinmar commented Feb 1, 2018

@Mgsy is it a regression? If not, is it possible to bisect it?

@Mgsy
Copy link
Member Author

Mgsy commented Feb 2, 2018

This is not a regression and I believe it occurs since the upload plugin has been introduced. After small investigation I can point some old commits:

  • Here is the first bad revision, which introduces the upload plugin to the editor - 22abf51.

  • And here is the first bad commit in the upload repository - ckeditor/ckeditor5-upload@9276e1d. This is the oldest one which allows to test this bug manually. Older ones don't have the insert image button, so testing this scenario is impossible.

This bug is easier to reproduce when you test it in the upload image manual test. My browser freezes every time after second d&d from the dialog. Still, I'm able to reproduce it only in Chrome on Windows.

@Reinmar Reinmar added this to the iteration 15 milestone Feb 2, 2018
@Reinmar Reinmar modified the milestones: iteration 15, iteration 16 Apr 5, 2018
@Reinmar Reinmar modified the milestones: iteration 18, next Jun 12, 2018
@FilipTokarski
Copy link
Member

FilipTokarski commented Jan 3, 2020

Checked it once again and it seems like Chrome is working fine now, no freezing.
However I'm not able to drag&drop the image in Firefox at all (only on Win).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
browser:chrome type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
Development

No branches or pull requests

4 participants