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

Chars are rendered incorrectly #4609

Closed
automatedbugreportingfacility opened this issue Apr 12, 2014 · 9 comments · Fixed by #4612
Closed

Chars are rendered incorrectly #4609

automatedbugreportingfacility opened this issue Apr 12, 2014 · 9 comments · Fixed by #4612

Comments

@automatedbugreportingfacility

This is a regression. Windows 7, ClearType disabled:

URL: http://www.upc.pl/pdf/upc_cennik_uslug_wazny_od_luty_2014.pdf
0.8.759: http://i.imgur.com/mucKg6C.png
1.0.2: http://i.imgur.com/4O8UEgz.png

@Snuffleupagus
Copy link
Collaborator

@automatedbugreportingfacility Would you be able to bisect where this regressed?
See https://github.com/mozilla/pdf.js/wiki/Bisecting-a-Regression.

@yurydelendik
Copy link
Contributor

@automatedbugreportingfacility I was unable to replicate the problem on Windows 7 ClearType disabled. Could you copy'n'paste the Graphics section from the the about:support?

@yurydelendik
Copy link
Contributor

Also, can you replicate the issue at http://mzl.la/pdf-js ?

@automatedbugreportingfacility
Copy link
Author

b5b94a4 is the first bad commit
commit b5b94a4
Author: Brendan Dahl
Date: Tue Feb 11 10:27:09 2014 -0800

Use built in CMaps and unify the glyph mapping.

@automatedbugreportingfacility
Copy link
Author

Could you copy'n'paste the Graphics section from the the about:support?

http://i.imgur.com/IYUeluq.png

Also, can you replicate the issue at http://mzl.la/pdf-js ?

Yes.

@yurydelendik
Copy link
Contributor

Hmm, DirectWrite 6.1 on Window 7. Problem is: it's almost impossible to find such configuration (IE10/11 updates to 6.2). I can only speculate that it's numGlyphs issue specified at #4573 (comment), no way for me to verify the fix atm

@automatedbugreportingfacility
Copy link
Author

Problem is: it's almost impossible to find such configuration (IE10/11 updates to 6.2).

IE10/11 requires a Windows platform update that in combination with my GPU driver causes rendering bugs in Firefox: https://bugzilla.mozilla.org/show_bug.cgi?id=812695

If you have a fix for this issue, I can verify it for you. Btw, the PDF in the bug you linked to is broken here too: http://i.imgur.com/Ag2iSCT.png

@yurydelendik
Copy link
Contributor

@automatedbugreportingfacility can you try version at #4612 ?

@automatedbugreportingfacility
Copy link
Author

It fixes the problem.

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

Successfully merging a pull request may close this issue.

4 participants