[pdf][thumbnails] Resize after move

ONLYOFFICE · Feb 11, 2025 · caecd26 · caecd26
1 parent a2436fb
commit caecd26
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 5 deletions.
diff --git a/pdf/src/document.js b/pdf/src/document.js
@@ -2391,8 +2391,11 @@ var CPresentation = CPresentation || function(){};
             for (let i = nStart; i <= nEnd; i++) {
                 oThumbnails._repaintPage(i);
             }
+
+            oThumbnails.setNeedResize(true);
         }
 
+        this.Viewer.resize(true);
         this.Viewer.paint();
         return true;
     };

diff --git a/pdf/src/history/documentChanges.js b/pdf/src/history/documentChanges.js
@@ -1125,7 +1125,10 @@ CChangesPDFDocumentMovePage.prototype.private_SetValue = function(nNewPos)
         for (let i = nStart; i <= nEnd; i++) {
             oThumbnails._repaintPage(i);
         }
+
+        oThumbnails.setNeedResize(true);
     }
 
+    oDoc.Viewer.resize(true);
     oDoc.Viewer.paint();
 };
diff --git a/pdf/src/thumbnails.js b/pdf/src/thumbnails.js
@@ -618,7 +618,11 @@
 
     // rendering
     CDocument.prototype._paint= function() {
-        if (!this.canvas) return;
+        if (!this.canvas || !this.viewer.canInteract()) return;
+        if (this.isNeedResize()) {
+            this.resize();
+        }
+
         let ctx= this.canvas.getContext("2d");
         this.canvas.width= this.canvas.width;
         ctx.fillStyle= ThumbnailsStyle.backgroundColor;
@@ -654,6 +658,12 @@
         this.resize();
     };
 
+    CDocument.prototype.setNeedResize = function(bResize) {
+        this.needResize = bResize;
+    };
+    CDocument.prototype.isNeedResize = function() {
+        return !!this.needResize;
+    };
     CDocument.prototype._resize = function(isZoomUpdated)
     {
         var element = document.getElementById(this.id);
@@ -769,6 +779,7 @@
 
         this.documentHeight = blockTop;
 
+        this.setNeedResize(false);
         this.updateScroll(scrollV);
         this.calculateVisibleBlocks();
         this.repaint();
@@ -1153,8 +1164,11 @@
         this.dragCanvas.height = h + 15;
         this.dragCanvas.style.display = "block";
 
+        // Set opacity
+        this.dragCanvas.style.opacity = 0.95;
+
         // Clear the canvas
-        this.dragCtx.clearRect(0, 0, w, h);
+        this.dragCtx.clearRect(0, 0, this.dragCanvas.width, this.dragCanvas.height);
 
         // If a stack of pages needs to be created
         if (countPages && countPages > 1) {
@@ -1185,9 +1199,6 @@
         }
         this.dragCtx.strokeRect(0, 0, w, h);
 
-        // Set opacity
-        this.dragCanvas.style.opacity = 0.95;
-
         // If more than one page is being dragged, display text with the number of pages
         if (countPages && countPages > 1) {
             let text = countPages + " pages";