Update thumbnail images only when sidebar is visible

mozilla · Jan 2, 2015 · 6f99d68 · 6f99d68
1 parent 0fa5017
commit 6f99d68
Showing 1 changed file with 18 additions and 5 deletions.
diff --git a/web/viewer.js b/web/viewer.js
@@ -1221,6 +1221,23 @@ var PDFViewerApplication = {
     }
   },
 
+  refreshSidebar: function pdfViewOpenSidebar() {
+    var pdfViewer = this.pdfViewer;
+    var thumbnailViewer = this.pdfThumbnailViewer;
+
+    thumbnailViewer.scrollThumbnailIntoView(this.page);
+
+    // set thumbnail images of rendered pages
+    var pagesCount = pdfViewer.pagesCount;
+    for (var pageIndex = 0; pageIndex < pagesCount; pageIndex++) {
+      var pageView = pdfViewer.getPageView(pageIndex);
+      if (pageView && pageView.renderingState === RenderingStates.FINISHED) {
+        var thumbnailView = thumbnailViewer.getThumbnail(pageIndex);
+        thumbnailView.setImage(pageView.canvas);
+      }
+    }
+  },
+
   switchSidebarView: function pdfViewSwitchSidebarView(view, openSidebar) {
     if (openSidebar && !this.sidebarOpen) {
       document.getElementById('sidebarToggle').click();
@@ -1621,8 +1638,7 @@ function webViewerInitialized() {
       PDFViewerApplication.sidebarOpen =
         outerContainer.classList.contains('sidebarOpen');
       if (PDFViewerApplication.sidebarOpen) {
-        PDFViewerApplication.pdfThumbnailViewer.
-          scrollThumbnailIntoView(PDFViewerApplication.page);
+        PDFViewerApplication.refreshSidebar();
       }
       PDFViewerApplication.forceRendering();
     });
@@ -1737,9 +1753,6 @@ document.addEventListener('DOMContentLoaded', webViewerLoad, true);
 document.addEventListener('pagerendered', function (e) {
   var pageIndex = e.detail.pageNumber - 1;
   var pageView = PDFViewerApplication.pdfViewer.getPageView(pageIndex);
-  var thumbnailView = PDFViewerApplication.pdfThumbnailViewer.
-                      getThumbnail(pageIndex);
-  thumbnailView.setImage(pageView.canvas);
 
 //#if (FIREFOX || MOZCENTRAL)
 //if (pageView.textLayer && pageView.textLayer.textDivs &&