From a3644888955c07adf57d48d25b4a4cf286e9c085 Mon Sep 17 00:00:00 2001
From: Lim Chee Aun <cheeaun@gmail.com>
Date: Sun, 1 Oct 2023 17:14:32 +0800
Subject: [PATCH] Test only use longpress for iOS

---
 src/components/status.jsx | 34 ++++++++++++++++++++++------------
 1 file changed, 22 insertions(+), 12 deletions(-)

diff --git a/src/components/status.jsx b/src/components/status.jsx
index 307d3885..4a7d6ebe 100644
--- a/src/components/status.jsx
+++ b/src/components/status.jsx
@@ -789,19 +789,29 @@ function Status({
     x: 0,
     y: 0,
   });
+  const isIOS =
+    window.ontouchstart !== undefined &&
+    /iPad|iPhone|iPod/.test(navigator.userAgent);
+  // Only iOS/iPadOS browsers don't support contextmenu
+  // Some comments report iPadOS might support contextmenu if a mouse is connected
   const bindLongPressContext = useLongPress(
-    (e) => {
-      const { clientX, clientY } = e.touches?.[0] || e;
-      // link detection copied from onContextMenu because here it works
-      const link = e.target.closest('a');
-      if (link && /^https?:\/\//.test(link.getAttribute('href'))) return;
-      e.preventDefault();
-      setContextMenuAnchorPoint({
-        x: clientX,
-        y: clientY,
-      });
-      setIsContextMenuOpen(true);
-    },
+    isIOS
+      ? (e) => {
+          if (e.pointerType === 'mouse') return;
+          // There's 'pen' too, but not sure if contextmenu event would trigger from a pen
+
+          const { clientX, clientY } = e.touches?.[0] || e;
+          // link detection copied from onContextMenu because here it works
+          const link = e.target.closest('a');
+          if (link && /^https?:\/\//.test(link.getAttribute('href'))) return;
+          e.preventDefault();
+          setContextMenuAnchorPoint({
+            x: clientX,
+            y: clientY,
+          });
+          setIsContextMenuOpen(true);
+        }
+      : null,
     {
       threshold: 600,
       captureEvent: true,