From ba376e17a9c3afb080728a67a8840572c24ceacc Mon Sep 17 00:00:00 2001
From: olaszakos <olaszakos@gmail.com>
Date: Thu, 4 Jan 2024 17:00:24 +0100
Subject: [PATCH] fix: mobile nav on ios could scroll its background

---
 src/components/Common/Search/Search.tsx | 17 +++++++++++++----
 src/theme/Layout/index.js               |  1 -
 src/utils/use-lock-body-scroll.ts       | 23 +++++++++++++++++++++++
 3 files changed, 36 insertions(+), 5 deletions(-)
 create mode 100644 src/utils/use-lock-body-scroll.ts

diff --git a/src/components/Common/Search/Search.tsx b/src/components/Common/Search/Search.tsx
index ecdb87835c..c4f1863043 100644
--- a/src/components/Common/Search/Search.tsx
+++ b/src/components/Common/Search/Search.tsx
@@ -6,6 +6,7 @@ import { PageSearchResult } from "@site/search/src/declarations/search/search.di
 import { createFocusTrap } from "focus-trap";
 import Link from "@docusaurus/Link";
 import { trackEvent } from "@site/src/utils/matomo";
+import useLockBodyScroll from "../../../utils/use-lock-body-scroll";
 
 let initialTerm = "";
 let initialResults: PageSearchResult[] | null = null;
@@ -26,6 +27,8 @@ const Search: FC<{ onClose: () => void }> = ({ onClose }) => {
   >({});
   const [loading, setLoading] = useState(false);
 
+  useLockBodyScroll(dialogRef);
+
   useEffect(() => {
     actorRefPromise.current = import("./actor").then(({ createActor }) => {
       console.log("search module loaded");
@@ -52,9 +55,7 @@ const Search: FC<{ onClose: () => void }> = ({ onClose }) => {
 
     window.addEventListener("keydown", onKeydown);
 
-    document.body.style.overflow = "hidden";
     return () => {
-      document.body.style.overflow = "unset";
       window.removeEventListener("keydown", onKeydown);
       trap.deactivate();
     };
@@ -104,7 +105,15 @@ const Search: FC<{ onClose: () => void }> = ({ onClose }) => {
   return (
     <>
       <motion.div
-        className="fixed inset-0 backdrop-blur-lg z-[3001] overflow-auto"
+        className="fixed h-screen inset-0 backdrop-blur-lg z-[3001] touch-none"
+        initial={{ opacity: 0 }}
+        animate={{ opacity: 1 }}
+        exit={{ opacity: 0 }}
+        onClick={onClose}
+        aria-modal="true"
+      ></motion.div>
+      <motion.div
+        className="fixed max-h-screen top-0 left-0 right-0 z-[3002] overflow-auto"
         initial={{ opacity: 0 }}
         animate={{ opacity: 1 }}
         exit={{ opacity: 0 }}
@@ -113,7 +122,7 @@ const Search: FC<{ onClose: () => void }> = ({ onClose }) => {
         aria-modal="true"
       >
         <div
-          className="md:container-10 backdrop-blur-0"
+          className="md:container-10 backdrop-blur-0 touch-auto"
           aria-label="Search the website"
         >
           <div
diff --git a/src/theme/Layout/index.js b/src/theme/Layout/index.js
index 41ff442c2b..f440a1f025 100644
--- a/src/theme/Layout/index.js
+++ b/src/theme/Layout/index.js
@@ -56,7 +56,6 @@ export default function Layout(props) {
         </ErrorBoundary>
       </div>
       {!noFooter && <Footer editPath={editPath} />}
-      <div id="search-overlay-portal"></div>
     </LayoutProvider>
   );
 }
diff --git a/src/utils/use-lock-body-scroll.ts b/src/utils/use-lock-body-scroll.ts
new file mode 100644
index 0000000000..2711b3badc
--- /dev/null
+++ b/src/utils/use-lock-body-scroll.ts
@@ -0,0 +1,23 @@
+import { useEffect } from "react";
+
+const useLockBodyScroll = (modalRef: React.RefObject<HTMLDivElement>) => {
+  useEffect(() => {
+    // Get original body overflow
+    const originalStyle = window.getComputedStyle(document.body).overflow;
+    const originalTouchAction = window.getComputedStyle(
+      document.body
+    ).touchAction;
+
+    // Prevent scrolling on mount
+    document.body.style.overflow = "hidden";
+    document.body.style.touchAction = "none";
+
+    // Re-enable scrolling when component unmounts
+    return () => {
+      document.body.style.overflow = originalStyle;
+      document.body.style.touchAction = originalTouchAction;
+    };
+  }, [modalRef]); // Empty array ensures effect is only run on mount and unmount
+};
+
+export default useLockBodyScroll;