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

font show error #3652

Closed
icode opened this issue Sep 6, 2013 · 9 comments · Fixed by #5651
Closed

font show error #3652

icode opened this issue Sep 6, 2013 · 9 comments · Fixed by #5651

Comments

@icode
Copy link

icode commented Sep 6, 2013

pdf show error

pdf file
http://pan.baidu.com/share/link?shareid=2977317873&uk=1543899842

@Snuffleupagus
Copy link
Collaborator

I can confirm the rendering issues on: Windows 7 (64-bit), Nightly 26 (buildID: 20130905030206) and PDF.js 0.8.495.
This is what the first page looks like on my computer:
issue3652

@timvandermeij
Copy link
Contributor

Uses Identity-H encoding for fonts. This has been a problem many times before, but I'm not sure if it also causes this particular problem.

/cc @yurydelendik

@timvandermeij
Copy link
Contributor

This is rendering a lot better now. No more random characters; most characters are actually being displayed now. Unfortunately, there are still some characters missing.

@brendandahl
Copy link
Contributor

How do you actually download the file? I keep getting a captcha that doesn't work.

@Snuffleupagus
Copy link
Collaborator

With #4259 applied, it's almost fixed, but it appears that some of glyphs are placed on top of each other:
3652_rendering

Compared with the rendering in Adobe Reader:
3652_rendering_adobe

@brendandahl I've previously managed to download the file, so it you want I can email it to you.

@timvandermeij
Copy link
Contributor

@brendandahl That's odd, the captcha does seem to work for me. However, maybe @Snuffleupagus has sent you the file already in the meantime :)

@timvandermeij
Copy link
Contributor

Will be fixed by #4259. Brendan has received the file in the meantime.

@Snuffleupagus
Copy link
Collaborator

Will be fixed by #4259.

Not completely! If you compare the screen-shots in my previous post (#3652 (comment)), you will see that with #4259, some of the glyphs are placed on top of each other.

@timvandermeij
Copy link
Contributor

I see, they should be shifted a bit left I think. Sorry for missing that!

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