From 4c25dcfb2bac5d3e0a36b58bd940dd5045ee0dea Mon Sep 17 00:00:00 2001
From: Alf Eaton <alf.eaton@overleaf.com>
Date: Fri, 15 Oct 2021 10:51:04 +0100
Subject: [PATCH] Disable overflow: scroll on the outer pdf.js element (#5446)

GitOrigin-RevId: 77485d0dd2af3e4dc6f86f322816e00beee9dfa9
---
 .../js/features/pdf-preview/components/pdf-js-viewer.js     | 2 +-
 services/web/frontend/stylesheets/app/editor/pdf.less       | 6 ++++++
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/services/web/frontend/js/features/pdf-preview/components/pdf-js-viewer.js b/services/web/frontend/js/features/pdf-preview/components/pdf-js-viewer.js
index e81860486b..7a3d60fd6e 100644
--- a/services/web/frontend/js/features/pdf-preview/components/pdf-js-viewer.js
+++ b/services/web/frontend/js/features/pdf-preview/components/pdf-js-viewer.js
@@ -237,7 +237,7 @@ function PdfJsViewer({ url }) {
   /* eslint-disable jsx-a11y/no-noninteractive-tabindex */
   /* eslint-disable jsx-a11y/no-noninteractive-element-interactions */
   return (
-    <div className="pdfjs-viewer" ref={handleContainer}>
+    <div className="pdfjs-viewer pdfjs-viewer-outer" ref={handleContainer}>
       <div
         className="pdfjs-viewer-inner"
         role="tabpanel"
diff --git a/services/web/frontend/stylesheets/app/editor/pdf.less b/services/web/frontend/stylesheets/app/editor/pdf.less
index 57c1e8f071..5a68c55088 100644
--- a/services/web/frontend/stylesheets/app/editor/pdf.less
+++ b/services/web/frontend/stylesheets/app/editor/pdf.less
@@ -268,6 +268,12 @@
   }
 }
 
+// The new viewer UI has overflow on the inner element,
+// so disable the overflow on the outer element
+.pdf-viewer .pdfjs-viewer.pdfjs-viewer-outer {
+  overflow: hidden;
+}
+
 .pdf-logs {
   overflow: auto;
   .alert {