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

Blurred PDFViewer text #6923

Closed
Dimitar-Goshev opened this issue Jul 13, 2022 · 1 comment
Closed

Blurred PDFViewer text #6923

Dimitar-Goshev opened this issue Jul 13, 2022 · 1 comment
Assignees
@tsvetomir @Iankodj
Labels
Bug C: PDFViewer FP: Completed Sync status with associated Feedback Item jQuery SEV: Medium
Milestone
2022.R2.SP2

Comments

@Dimitar-Goshev
Copy link
Contributor

Bug report

Text of PDF documents in PDFViewer may be blurred.

Reproduction of the problem

  1. Go to PDFViewer demo or run the following Dojo snippet

Current behavior

Text is blurred, most visible on small letters.

Expected/desired behavior

Text is not blurred.

Environment

  • Kendo UI version: 2022.2.621
  • Browser: [Chrome 103 | Firefox 102| Edge 102]
@Iankodj
Copy link
Contributor

Iankodj commented Jul 14, 2022 •
edited
Loading

Suggested fix:

the canvas content is upscaled since we have more than physical 1 pixel per CSS pixel
the fix is to render a larger canvas, but keep it's layout dimensions the same
canvas.width should be element.offsetWidth * window.devicePixelRatio
same for height
https://developer.mozilla.org/en-US/docs/Web/API/Canvas_API/Tutorial/Optimizing_canvas#scaling_for_high_resolution_displays

@kendo-bot kendo-bot added FP: Planned Sync status with associated Feedback Item FP: In Development Sync status with associated Feedback Item and removed FP: Planned Sync status with associated Feedback Item labels Jul 19, 2022
@Dimitar-Goshev Dimitar-Goshev added this to the 2022.R2.SP.next milestone Jul 19, 2022
@kendo-bot kendo-bot added FP: Completed Sync status with associated Feedback Item and removed FP: In Development Sync status with associated Feedback Item labels Jul 19, 2022
@Dimitar-Goshev Dimitar-Goshev mentioned this issue Mar 2, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tsvetomir tsvetomir

@Iankodj Iankodj

Labels
Bug C: PDFViewer FP: Completed Sync status with associated Feedback Item jQuery SEV: Medium
Projects
None yet
Milestone
2022.R2.SP2
Development

No branches or pull requests
4 participants
@tsvetomir @kendo-bot @Iankodj @Dimitar-Goshev