From 2a6b84596e257fa81967881aa28c73b1d1b8ed31 Mon Sep 17 00:00:00 2001
From: David <33458145+davidmcpowell@users.noreply.github.com>
Date: Thu, 20 Jun 2024 13:12:01 +0100
Subject: [PATCH] Merge pull request #19006 from overleaf/dp-pdf-click

Add click listener to pdf presentation mode to go to next page

GitOrigin-RevId: add7ddb18ef63774bbb78f6337088bd59b087db1
---
 .../hooks/use-presentation-mode.ts            | 71 +++++++++++++------
 1 file changed, 49 insertions(+), 22 deletions(-)

diff --git a/services/web/frontend/js/features/pdf-preview/hooks/use-presentation-mode.ts b/services/web/frontend/js/features/pdf-preview/hooks/use-presentation-mode.ts
index e2cb859292..457704da00 100644
--- a/services/web/frontend/js/features/pdf-preview/hooks/use-presentation-mode.ts
+++ b/services/web/frontend/js/features/pdf-preview/hooks/use-presentation-mode.ts
@@ -18,42 +18,69 @@ export default function usePresentationMode(
 
   const [presentationMode, setPresentationMode] = useState(false)
 
-  const arrowKeyListener = useCallback(
+  const nextPage = useCallback(() => {
+    if (page !== null) {
+      handlePageChange(page + 1)
+    }
+  }, [handlePageChange, page])
+
+  const previousPage = useCallback(() => {
+    if (page !== null) {
+      handlePageChange(page - 1)
+    }
+  }, [handlePageChange, page])
+
+  const clickListener = useCallback(
     event => {
-      if (page !== null) {
-        switch (event.key) {
-          case 'ArrowLeft':
-          case 'ArrowUp':
-            handlePageChange(page - 1)
-            break
+      if (event.target.tagName === 'A') {
+        return
+      }
 
-          case 'ArrowRight':
-          case 'ArrowDown':
-            handlePageChange(page + 1)
-            break
-
-          case ' ':
-            if (event.shiftKey) {
-              handlePageChange(page - 1)
-            } else {
-              handlePageChange(page + 1)
-            }
-            break
-        }
+      if (event.shiftKey) {
+        previousPage()
+      } else {
+        nextPage()
       }
     },
-    [page, handlePageChange]
+    [nextPage, previousPage]
+  )
+
+  const arrowKeyListener = useCallback(
+    event => {
+      switch (event.key) {
+        case 'ArrowLeft':
+        case 'ArrowUp':
+          previousPage()
+          break
+
+        case 'ArrowRight':
+        case 'ArrowDown':
+          nextPage()
+          break
+
+        case ' ':
+          if (event.shiftKey) {
+            previousPage()
+          } else {
+            nextPage()
+          }
+          break
+      }
+    },
+    [nextPage, previousPage]
   )
 
   useEffect(() => {
     if (presentationMode) {
       window.addEventListener('keydown', arrowKeyListener)
+      window.addEventListener('click', clickListener)
 
       return () => {
         window.removeEventListener('keydown', arrowKeyListener)
+        window.removeEventListener('click', clickListener)
       }
     }
-  }, [presentationMode, arrowKeyListener])
+  }, [presentationMode, arrowKeyListener, clickListener])
 
   const requestPresentationMode = useCallback(() => {
     if (pdfJsWrapper) {