From b7c05f25f996405978b3dd8c8d1cfe65b66e8e3a Mon Sep 17 00:00:00 2001
From: Jordan Ribbink <jribbink@telus.net>
Date: Sun, 29 Sep 2024 18:54:35 -0700
Subject: [PATCH 1/2] Fix deep linking within in-app browsers

---
 .../views/ScanConnect/ScanConnectMobile.tsx   | 25 +++++++++++++------
 1 file changed, 17 insertions(+), 8 deletions(-)

diff --git a/components/views/ScanConnect/ScanConnectMobile.tsx b/components/views/ScanConnect/ScanConnectMobile.tsx
index 98595ac..4d425e6 100644
--- a/components/views/ScanConnect/ScanConnectMobile.tsx
+++ b/components/views/ScanConnect/ScanConnectMobile.tsx
@@ -20,12 +20,8 @@ export default function ScanConnectMobile({
   onGetWallet,
   noDeepLink,
 }: ScanConnectMobileProps) {
-  const windowRef = useRef<Window | null>(null)
   const { setLastUsed } = useWalletHistory()
   const { uri, connecting, error, isLoading } = useWcUri(() => {
-    if (windowRef.current && !windowRef.current.closed) {
-      windowRef.current.close()
-    }
     setLastUsed(wallet)
     handleCancel()
   })
@@ -36,12 +32,25 @@ export default function ScanConnectMobile({
   )
 
   function openDeepLink() {
-    if (windowRef.current && !windowRef.current.closed) {
-      windowRef.current.close()
-    }
     const deeplink = new URL(service.uid)
     deeplink.searchParams.set('uri', uri)
-    windowRef.current = window.open(deeplink, '_blank')
+
+    if (deeplink.toString().startsWith('http')) {
+      // Workaround for https://github.com/rainbow-me/rainbowkit/issues/524.
+      // Using 'window.open' causes issues on iOS in non-Safari browsers and
+      // WebViews where a blank tab is left behind after connecting.
+      // This is especially bad in some WebView scenarios (e.g. following a
+      // link from Twitter) where the user doesn't have any mechanism for
+      // closing the blank tab.
+      // For whatever reason, links with a target of "_blank" don't suffer
+      // from this problem, and programmatically clicking a detached link
+      // element with the same attributes also avoids the issue.
+      const link = document.createElement('a')
+      link.href = deeplink.toString()
+      link.target = '_blank'
+      link.rel = 'noreferrer noopener'
+      link.click()
+    }
   }
 
   useEffect(() => {

From d07f3722333cc2fa16f9d519f750729be2d69c2f Mon Sep 17 00:00:00 2001
From: Jordan Ribbink <jribbink@telus.net>
Date: Sun, 29 Sep 2024 19:15:39 -0700
Subject: [PATCH 2/2] add missing else

---
 components/views/ScanConnect/ScanConnectMobile.tsx | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/components/views/ScanConnect/ScanConnectMobile.tsx b/components/views/ScanConnect/ScanConnectMobile.tsx
index 4d425e6..e49f09b 100644
--- a/components/views/ScanConnect/ScanConnectMobile.tsx
+++ b/components/views/ScanConnect/ScanConnectMobile.tsx
@@ -50,6 +50,8 @@ export default function ScanConnectMobile({
       link.target = '_blank'
       link.rel = 'noreferrer noopener'
       link.click()
+    } else {
+      window.location.href = deeplink.toString()
     }
   }