Move AssetListTab spans end calls into branches in useEffect hooks

MetaMask · Nov 5, 2024 · 0d5b194 · 0d5b194
1 parent bff8cbe
commit 0d5b194
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/ui/components/app/assets/token-list/token-list.tsx b/ui/components/app/assets/token-list/token-list.tsx
@@ -67,12 +67,12 @@ export default function TokenList({
   ]);
 
   useEffect(() => {
-    endTrace({ name: TraceName.AccountOverviewAssetListTabFCP });
-  }, []);
-
-  if (!loading) {
-    endTrace({ name: TraceName.AccountOverviewAssetListTab });
-  }
+    if (loading) {
+      endTrace({ name: TraceName.AccountOverviewAssetListTabFCP });
+    } else {
+      endTrace({ name: TraceName.AccountOverviewAssetListTab });
+    }
+  }, [loading]);
 
   return loading ? (
     <Box