From ae606064703d13b6d1fbb27bfe72d1be40d0dea2 Mon Sep 17 00:00:00 2001
From: Huang Xin <chrox.huang@gmail.com>
Date: Fri, 24 Jan 2025 23:11:05 +0100
Subject: [PATCH] mobile: disable contextmenu only for mobile devices (#243)

---
 .../app/reader/components/annotator/Annotator.tsx | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/apps/readest-app/src/app/reader/components/annotator/Annotator.tsx b/apps/readest-app/src/app/reader/components/annotator/Annotator.tsx
index f76ba485..91f18982 100644
--- a/apps/readest-app/src/app/reader/components/annotator/Annotator.tsx
+++ b/apps/readest-app/src/app/reader/components/annotator/Annotator.tsx
@@ -128,11 +128,16 @@ const Annotator: React.FC<{ bookKey: string }> = ({ bookKey }) => {
       detail.doc?.addEventListener('pointerup', handlePointerup);
       detail.doc?.addEventListener('touchmove', handleTouchmove);
       detail.doc?.addEventListener('selectionchange', handleSelectionchange);
-      detail.doc?.addEventListener('contextmenu', (event: Event) => {
-        event.preventDefault();
-        event.stopPropagation();
-        return false;
-      });
+
+      // Disable the default context menu on mobile devices,
+      // although it should but doesn't work on iOS
+      if (osPlatform === 'android' || osPlatform === 'ios') {
+        detail.doc?.addEventListener('contextmenu', (event: Event) => {
+          event.preventDefault();
+          event.stopPropagation();
+          return false;
+        });
+      }
     }
   };