chore: don't show account info and balance in psbt headers and block …

…logo click
leather-io · Mar 21, 2024 · 927ee6f · 927ee6f
1 parent 91403ce
commit 927ee6f
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 7 deletions.
diff --git a/src/app/features/container/container.tsx b/src/app/features/container/container.tsx
@@ -33,6 +33,7 @@ import { TotalBalance } from './total-balance';
 import {
   getDisplayAddresssBalanceOf,
   isKnownPopupRoute,
+  isRpcRoute,
   showAccountInfo,
   showBalanceInfo,
 } from './utils/get-popup-header';
@@ -103,6 +104,7 @@ export function Container() {
   const hideSettings =
     isKnownPopupRoute(pathname) || isSummaryPage(pathname) || variant === 'onboarding';
 
+  const isLogoClickable = variant !== 'home' && !isRpcRoute(pathname);
   return (
     <>
       <SwitchAccountDialog
@@ -149,7 +151,7 @@ export function Container() {
                   >
                     <Logo
                       data-testid={OnboardingSelectors.LogoRouteToHome}
-                      onClick={variant !== 'home' ? () => navigate(RouteUrls.Home) : undefined}
+                      onClick={isLogoClickable ? () => navigate(RouteUrls.Home) : undefined}
                     />
                   </Box>
                 )

diff --git a/src/app/features/container/utils/get-popup-header.ts b/src/app/features/container/utils/get-popup-header.ts
@@ -3,14 +3,22 @@
  */
 import { RouteUrls } from '@shared/route-urls';
 
+export function isRpcRoute(pathname: RouteUrls) {
+  switch (pathname) {
+    case RouteUrls.PsbtRequest:
+    case RouteUrls.SignatureRequest:
+    case RouteUrls.RpcStacksSignature:
+      return true;
+    default:
+      return false;
+  }
+}
+
 export function showAccountInfo(pathname: RouteUrls) {
   switch (pathname) {
     case RouteUrls.TransactionRequest:
     case RouteUrls.ProfileUpdateRequest:
-    case RouteUrls.PsbtRequest:
     case RouteUrls.RpcSendTransfer:
-    case RouteUrls.SignatureRequest:
-    case RouteUrls.RpcStacksSignature:
       return true;
     default:
       return false;
@@ -20,9 +28,7 @@ export function showAccountInfo(pathname: RouteUrls) {
 export function showBalanceInfo(pathname: RouteUrls) {
   switch (pathname) {
     case RouteUrls.ProfileUpdateRequest:
-    case RouteUrls.PsbtRequest:
     case RouteUrls.RpcSendTransfer:
-    case RouteUrls.RpcStacksSignature:
       return true;
     case RouteUrls.TransactionRequest:
     default:
@@ -35,7 +41,6 @@ export function getDisplayAddresssBalanceOf(pathname: RouteUrls) {
   switch (pathname) {
     case RouteUrls.TransactionRequest:
     case RouteUrls.ProfileUpdateRequest:
-    case RouteUrls.PsbtRequest:
     case RouteUrls.RpcSendTransfer:
       return 'all';
     case RouteUrls.RpcStacksSignature: