fix: update popup headers to show account info, ref leather-io/issues…

…#157
leather-io · Jul 24, 2024 · 93ad45f · 93ad45f
1 parent 95d6ea5
commit 93ad45f
Showing 1 changed file with 11 additions and 6 deletions.
diff --git a/src/app/features/container/utils/get-popup-header.ts b/src/app/features/container/utils/get-popup-header.ts
@@ -1,6 +1,3 @@
-/**
- * POPUP header logic notes here -> https://github.com/leather-io/extension/issues/4371#issuecomment-1919114939
- */
 import { RouteUrls } from '@shared/route-urls';
 
 export function isRpcRoute(pathname: RouteUrls) {
@@ -24,11 +21,14 @@ export function isRpcRoute(pathname: RouteUrls) {
 
 export function showAccountInfo(pathname: RouteUrls) {
   switch (pathname) {
+    case RouteUrls.PsbtRequest:
     case RouteUrls.TransactionRequest:
     case RouteUrls.ProfileUpdateRequest:
     case RouteUrls.RpcSendTransfer:
     case RouteUrls.RpcSignPsbt:
     case RouteUrls.RpcSignBip322Message:
+    case RouteUrls.SignatureRequest:
+    case RouteUrls.RpcStacksSignature:
       return true;
     default:
       return false;
@@ -39,20 +39,25 @@ export function showBalanceInfo(pathname: RouteUrls) {
   switch (pathname) {
     case RouteUrls.ProfileUpdateRequest:
     case RouteUrls.RpcSendTransfer:
-      return true;
+    case RouteUrls.PsbtRequest:
+    case RouteUrls.RpcSignBip322Message:
+    case RouteUrls.RpcStacksSignature:
     case RouteUrls.TransactionRequest:
+      return true;
     default:
       return false;
   }
 }
 
 export function getDisplayAddresssBalanceOf(pathname: RouteUrls) {
-  //  TODO it's unclear when to show ALL or STX balance here
   switch (pathname) {
-    case RouteUrls.TransactionRequest:
     case RouteUrls.ProfileUpdateRequest:
     case RouteUrls.RpcSendTransfer:
+    case RouteUrls.PsbtRequest:
+    case RouteUrls.RpcSignBip322Message:
       return 'all';
+    case RouteUrls.RpcStacksSignature:
+    case RouteUrls.TransactionRequest:
     default:
       return 'stx';
   }