PDF fuzzy on retina about angularjs-pdf HOT 9 CLOSED

wow! thanks - yes please send a PDF 👍

from angularjs-pdf.

I assume by PDF you mean PR? 😛

I will try to clean it up and submit something this week.

from angularjs-pdf.

LOL yes, please send a pull request 😝

from angularjs-pdf.

I actually looked at my retina support commit, and it might be good as is. I submitted a PR. Please let me know your thoughts and I'll be glad to make any necessary changes.

from angularjs-pdf.

Hi guys, I have noticed that this PR results in a blurry canvas due to the dimensions being set to decimal values e.g. 1024.3456. When both the canvas dimensions and the style dimensions are set to this value the canvas dimensions appear to be truncated but the style dimensions will stay as a number containing decimals and so they won't match - this results in blurring (tested on 1080p monitor in Chrome and Firefox).

I have patched this locally by changing the following lines within the new setCanvasDimensions() method:

canvas.style.width = w + 'px';
canvas.style.height = h + 'px';

to:

canvas.style.width = Math.floor(w) + 'px';
canvas.style.height = Math.floor(h) + 'px';

With this patch the canvas appears normally - note I haven't tested on a retina screen as I don't have one with me at the moment.

from angularjs-pdf.

Thanks @ElRoberto538! Could you send us a pull request to the changes in this file example/js/directives/angular-pdf.js? Also do you have a sample pdf page?

I have a retina screen that I can test on.

from angularjs-pdf.

No problem, I will create a pull request later today - I will also find a suitable sample PDF to demonstrate the fix.

from angularjs-pdf.

Good catch @ElRoberto538. I have a retina screen, I'd be glad to check it out. Let me know when a PR is up.

from angularjs-pdf.

@rwwagner90 - it's here: #56

thanks everyone 🌟

from angularjs-pdf.