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.

Sign up for GitHub

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

Jump to bottom

Image dialog opened for image inside table for the second time does not retain image data correctly #3167

Open
f1ames opened this issue Jun 14, 2019 · 0 comments
Open

Image dialog opened for image inside table for the second time does not retain image data correctly #3167

f1ames opened this issue Jun 14, 2019 · 0 comments
Labels
browser:ie11 The issue can only be reproduced in the Internet Explorer 11 browser. plugin:image The plugin which probably causes the issue. plugin:table The plugin which probably causes the issue. status:confirmed An issue confirmed by the development team. type:bug A bug.

Comments

@f1ames
Copy link
Contributor

f1ames commented Jun 14, 2019

Type of report

Bug

Provide detailed reproduction steps (if any)

  1. Go to https://jsfiddle.net/40de2zrb/2/embedded/result,css,html,js.
  2. Double click image inside table.
    • Image dialog with correct image data is opened.
  3. Double click on image or open image dialog via context menu.

ie11 nots

Expected result

Image dialog with correct image data is opened.

Actual result

Image dialog is missing image data.

Other details

Could be related to #3166 (I suspect fixing this one may fix #3166).

  • Browser: IE11 (works fine on Chrome, FF and Edge)
  • OS: Window 7/10
  • CKEditor version: 4.10.0/4.11.4
  • Installed CKEditor plugins: full preset without table selection
@f1ames f1ames added type:bug A bug. browser:ie11 The issue can only be reproduced in the Internet Explorer 11 browser. plugin:image The plugin which probably causes the issue. plugin:table The plugin which probably causes the issue. target:minor Any docs related issue that can be merged into a master or major branch. labels Jun 14, 2019
@f1ames f1ames mentioned this issue Jun 14, 2019
Merged
2 tasks
@jacekbogdanski jacekbogdanski added the status:confirmed An issue confirmed by the development team. label Jun 22, 2019
@f1ames f1ames removed the target:minor Any docs related issue that can be merged into a master or major branch. label Nov 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
browser:ie11 The issue can only be reproduced in the Internet Explorer 11 browser. plugin:image The plugin which probably causes the issue. plugin:table The plugin which probably causes the issue. status:confirmed An issue confirmed by the development team. type:bug A bug.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@f1ames @jacekbogdanski