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

Wrong resize/rounding-math. #1732

Open
hrach opened this issue May 21, 2012 · 7 comments
Open

Wrong resize/rounding-math. #1732

hrach opened this issue May 21, 2012 · 7 comments

Comments

@hrach
Copy link

hrach commented May 21, 2012

PDF: http://www.3pe.cz/PDFnoty/Chvalte%20Hospodina%20z%20nebes.pdf
Screenshot: http://cl.ly/image/080v2f263X1G

Becasue the note-line is badly drawed, it seems the first tone is F, but it's G.
In zoom 200% it's ok. I understand difficulties in smaller size, but 100% zoom should seemed fine.

FF 15a1, win 7 x64, gpu draw enabled, pdf.js 0.2.536

@saebekassebil
Copy link
Contributor

Works for me (FF20, Mac OSX), @hrach - Is this still an issue with latest PDF.js extension?

@hrach
Copy link
Author

hrach commented Apr 15, 2013

I installed 0.8.49 into my Firefox 21 (beta). Rendered image is still wrong. I have no idea if I have to adjust Firefox to use installed oextension over the included one, but sadly still misredenred.

Win 8 x64, Fx 21b, gpu enabled, pdf.js 0.8.64

@timvandermeij
Copy link
Contributor

I can reproduce this using Windows 7 x64, Firefox Nightly 30.0a1 and the latest PDF.js development version.

@himanshugarg
Copy link

himanshugarg commented Sep 23, 2021

[Newbie Alert] As the first note is centered on the line and not in the space between lines, it is a G. So this may no longer be an issue. Win 10, FF 92

@hrach
Copy link
Author

hrach commented Sep 23, 2021

Having 100% zoom renders this:
image
I'd guess this is still not fixed :/

@himanshugarg
Copy link

Thanks. This is how it looks for me on Ubuntu 16, FF 88.

image

Newbie question: do I have to install pdf.js? I am assuming it comes bundled with FF.

@hrach
Copy link
Author

hrach commented Sep 23, 2021

image
This seems rather to be F than G. So you are affected too. PDF.js is bundled.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants