Skip to content

Commit

Permalink
improve comment explaining standard fonts
Browse files Browse the repository at this point in the history
  • Loading branch information
thestinger committed Feb 18, 2025
1 parent 5145c03 commit 29feb73
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions viewer/js/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -400,9 +400,12 @@ globalThis.loadDocument = function () {
cMapPacked: true,
password: pdfPassword,
isEvalSupported: false,
// If font not embedded in PDF, the viewer falls back to default system fonts. Apparently
// on Android, won't be able to find a good substitution.
// https://github.com/mozilla/pdf.js/pull/18465, https://bugzilla.mozilla.org/show_bug.cgi?id=1882613
// If a font isn't embedded, the viewer falls back to default system fonts. On Android,
// there often isn't a good substitution provided by the OS, so we need to bundle standard
// fonts to improve the rendering of certain PDFs:
//
// https://github.com/mozilla/pdf.js/pull/18465
// https://bugzilla.mozilla.org/show_bug.cgi?id=1882613
useSystemFonts: false,
standardFontDataUrl: "https://localhost/standard_fonts/"
});
Expand Down

0 comments on commit 29feb73

Please sign in to comment.