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

Link and image are not displayed properly by print plugin #3876

Closed
Dumluregn opened this issue Feb 20, 2020 · 3 comments · Fixed by #4813
Closed

Link and image are not displayed properly by print plugin #3876

Dumluregn opened this issue Feb 20, 2020 · 3 comments · Fixed by #4813
Assignees
@sculpt0r
Labels
status:blocked An issue which development is blocked by another issue (internal or external one). status:confirmed An issue confirmed by the development team. type:bug A bug. type:failingtest A failing test.

Comments

@Dumluregn
Copy link
Contributor

Type of report

Bug

Provide detailed reproduction steps (if any)

  1. Go to /tests/plugins/print/manual/print
  2. Click Print button
  3. Check out preview

Expected result

Preview content matches editor content exactly.

Actual result

9/10 tries image is missing and link is displayed as plain text.

Other details

In Edge Chromium it works fine if you click Preview button first and don't close the preview window. This workaround doesn't work for Chrome though. Also it seemed to work in 4.13.1 (check it out here).

  • Browser: Chrome, Edge Chromium
  • OS: macOS
  • CKEditor version: 4.14.0
  • Installed CKEditor plugins: print, preview
@Dumluregn Dumluregn added type:bug A bug. status:confirmed An issue confirmed by the development team. regression This issue is a regression. labels Feb 20, 2020
@Dumluregn Dumluregn changed the title Anchor is not converted properly in print plugin Link and image are not displayed properly by print plugin Feb 20, 2020
@Dumluregn
Copy link
Contributor Author

I checked this also during monkey clicking in full preset and it works correctly there 🤔 So I'm relabelling from regression to failingtest.

@Dumluregn Dumluregn added type:failingtest A failing test. and removed regression This issue is a regression. labels Feb 20, 2020
@sculpt0r sculpt0r self-assigned this Aug 26, 2021
@sculpt0r sculpt0r mentioned this issue Aug 26, 2021
Merged
3 tasks
@sculpt0r sculpt0r linked a pull request Aug 26, 2021 that will close this issue
Fix fire print on Firefox #4813
Merged
3 tasks
@sculpt0r
Copy link
Contributor

sculpt0r commented Aug 26, 2021 •
edited
Loading

May be fixed by: #4813 . Marking this one as blocked until mentioned PR will be merged. On #4813 PR branch this test seems to be working fine 👍 So I'm marking this as blocked by this PR.

@sculpt0r sculpt0r added the status:blocked An issue which development is blocked by another issue (internal or external one). label Aug 26, 2021
@Comandeer
Copy link
Member

Closed by #4813.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sculpt0r sculpt0r

Labels
status:blocked An issue which development is blocked by another issue (internal or external one). status:confirmed An issue confirmed by the development team. type:bug A bug. type:failingtest A failing test.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix fire print on Firefox ckeditor/ckeditor4
3 participants
@Comandeer @sculpt0r @Dumluregn