pdf: fix getAnnotations issue

Failed to render 10.1007/978-3-662-54809-7 otherwise. Could fix it quickly thanks to mozilla/pdf.js#8429.
paperhive · Oct 22, 2017 · 3f344d2 · 3f344d2
1 parent aeec165
commit 3f344d2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/app/directives/pdf.ts b/app/directives/pdf.ts
@@ -297,7 +297,7 @@ export default function(app) {
         });
 
         if (!this.annotations) {
-          this.annotations = await this.page.getAnnotations();
+          this.annotations = await this.page.getAnnotations({intent: 'display'});
         }
 
         // wipe all children from the container

diff --git a/types/pdfjs.d.ts b/types/pdfjs.d.ts
@@ -214,13 +214,17 @@ interface PDFAnnotationLayerParameters {
   update(params: PDFAnnotationLayerParameters): void;
 }
 
+interface PDFGetAnnotationsParameters {
+  intent?: 'display' | 'print';
+}
+
 interface PDFPageProxy {
   pageNumber: number;
   rotate: number;
   ref: PDFRef;
   view: number[];
   getViewport(scale: number, rotate?: number): PDFPageViewport;
-  getAnnotations(): PDFPromise<PDFAnnotations>;
+  getAnnotations(params: PDFGetAnnotationsParameters?): PDFPromise<PDFAnnotations>;
   render(params: PDFRenderParams): PDFRenderTask;
   getTextContent(): PDFPromise<PDFTextContent>;
   // getOperationList(): PDFPromise<>;