fix: icon display in connect modal

MetaMask · Mar 6, 2024 · 4514277 · 4514277
1 parent ba3e052
commit 4514277
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/app/components/Views/AccountConnect/AccountConnect.tsx b/app/components/Views/AccountConnect/AccountConnect.tsx
@@ -102,8 +102,14 @@ const AccountConnect = (props: AccountConnectProps) => {
   // Last wallet connect session metadata
   const wc2Metadata = useSelector((state: RootState) => state.sdk.wc2Metadata);
 
+  const dappIconUrl = sdkConnection.originatorInfo?.icon;
   const faviconSource = useFavicon(origin);
 
+  const actualIcon = useMemo(
+    () => ({ uri: dappIconUrl } ?? faviconSource),
+    [dappIconUrl, faviconSource],
+  );
+
   const secureIcon = useMemo(
     () =>
       (getUrlObj(hostname) as URLParse<string>).protocol === 'https:'
@@ -363,7 +369,7 @@ const AccountConnect = (props: AccountConnectProps) => {
         onUserAction={setUserIntent}
         defaultSelectedAccount={defaultSelectedAccount}
         isLoading={isLoading}
-        favicon={faviconSource}
+        favicon={actualIcon}
         secureIcon={secureIcon}
         urlWithProtocol={urlWithProtocol}
       />
@@ -375,7 +381,7 @@ const AccountConnect = (props: AccountConnectProps) => {
     isLoading,
     setScreen,
     setSelectedAddresses,
-    faviconSource,
+    actualIcon,
     secureIcon,
     sdkConnection,
     urlWithProtocol,