fix: apply useSystemFonts: true by default

unjs · Nov 9, 2023 · 02afd71 · 02afd71
1 parent 61fb4f8
commit 02afd71
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/src/image.ts b/src/image.ts
@@ -107,7 +107,9 @@ async function createIsomorphicCanvasFactory(
         canvas.width = width;
         canvas.height = height;
         return canvas;
-      } else if (isNode) {
+      }
+
+      if (isNode) {
         if (!_canvas) {
           throw new Error('Failed to resolve "canvas" package.');
         }

diff --git a/src/pdfjs-serverless/rollup/plugins.ts b/src/pdfjs-serverless/rollup/plugins.ts
@@ -3,7 +3,7 @@ import type { Plugin } from "rollup";
 
 export function pdfjsTypes(): Plugin {
   return {
-    name: "pdfjs:types",
+    name: "pdfjs-serverless:types",
     async writeBundle() {
       const data = `
 import * as PDFJS from './types/src/pdf'

diff --git a/src/utils.ts b/src/utils.ts
@@ -15,6 +15,7 @@ export const isBrowser = typeof window !== "undefined";
  *
  * Applies the following defaults:
  * - `isEvalSupported: false`
+ * - `useSystemFonts: true`
  */
 export async function getDocumentProxy(
   data: BinaryData,
@@ -24,6 +25,8 @@ export async function getDocumentProxy(
   const pdf = await getDocument({
     data,
     isEvalSupported: false,
+    // See: https://github.com/mozilla/pdf.js/issues/4244#issuecomment-1479534301
+    useSystemFonts: true,
     ...options,
   }).promise;