Format 4 cmap are incorrects and are rejected by browser's font sanitizer #43

Closed
wants to merge 1 commit into from

2 participants

@vingtetun

With this the format 4 cmap is correct and the font sanitizer is happy.

This patch partially fix https://github.com/andreasgal/pdf.js/issues/560, the embedded font is used but they looks weird (that's something on the side of pdf.js).

@devongovett
Owner

Wow, thanks for the fix!

So let me just make sure I understand... This makes the fonts pass the sanitizer and the rest of the issues are on your side?

Here is what I'm getting with pdf.js after the change:
screenshot

Looks like we got correct glyphs in the wrong places now, i.e. "S" in place of "!" and "r" in place of ":", but at least the glyphs are loading at all! :)

@vingtetun
@devongovett
Owner

Hmm, good to know. I'll have to look into that. Thanks!

@vingtetun vingtetun closed this Oct 14, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment