From 1f083db66c05ed737d3ca9ccf037d7de2a62f00e Mon Sep 17 00:00:00 2001 From: David <33458145+davidmcpowell@users.noreply.github.com> Date: Mon, 3 Jun 2024 10:20:57 +0100 Subject: [PATCH] Merge pull request #18601 from overleaf/dp-pdf Only set initial pdf page number on first page render GitOrigin-RevId: f272acb2bce704037079e01dde89e475414e9fea --- .../js/features/pdf-preview/components/pdf-js-viewer.jsx | 3 +++ 1 file changed, 3 insertions(+) diff --git a/services/web/frontend/js/features/pdf-preview/components/pdf-js-viewer.jsx b/services/web/frontend/js/features/pdf-preview/components/pdf-js-viewer.jsx index fdc1cb76ca..e8ab562b6b 100644 --- a/services/web/frontend/js/features/pdf-preview/components/pdf-js-viewer.jsx +++ b/services/web/frontend/js/features/pdf-preview/components/pdf-js-viewer.jsx @@ -131,6 +131,9 @@ function PdfJsViewer({ url, pdfFile }) { const handleRenderedInitialPageNumber = () => { setPage(pdfJsWrapper.viewer.currentPageNumber) + + // Only need to set the initial page number once. + pdfJsWrapper.eventBus.off('pagerendered', handleRenderedInitialPageNumber) } const handleScaleChanged = () => {