feat: Hide collect button if there isn't any position to collect

sushi-labs · Jan 23, 2025 · 3152e14 · 3152e14
1 parent d1e2da5
commit 3152e14
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 23 deletions.
diff --git a/apps/web/src/ui/pool/ConcentratedLiquidityCollectAllWidget.tsx b/apps/web/src/ui/pool/ConcentratedLiquidityCollectAllWidget.tsx
@@ -224,28 +224,30 @@ export const ConcentratedLiquidityCollectAllWidget: FC<
       <DialogReview>
         {({ confirm }) => (
           <>
-            <div className="ml-auto">
-              <Checker.Connect className="min-w-[160px]" size="sm" fullWidth>
-                <Checker.Network
-                  className="min-w-[160px]"
-                  size="sm"
-                  fullWidth
-                  chainId={chainId}
-                >
-                  <DialogTrigger asChild>
-                    <Button
-                      className="min-w-[160px]"
-                      size="sm"
-                      fullWidth
-                      disabled={!aggregatedAmounts.length}
-                      testId="claim-fees-all"
-                    >
-                      Claim Fees
-                    </Button>
-                  </DialogTrigger>
-                </Checker.Network>
-              </Checker.Connect>
-            </div>
+            {aggregatedAmounts.length > 0 ? (
+              <div className="ml-auto">
+                <Checker.Connect className="min-w-[160px]" size="sm" fullWidth>
+                  <Checker.Network
+                    className="min-w-[160px]"
+                    size="sm"
+                    fullWidth
+                    chainId={chainId}
+                  >
+                    <DialogTrigger asChild>
+                      <Button
+                        className="min-w-[160px]"
+                        size="sm"
+                        fullWidth
+                        disabled={!aggregatedAmounts.length}
+                        testId="claim-fees-all"
+                      >
+                        Claim Fees
+                      </Button>
+                    </DialogTrigger>
+                  </Checker.Network>
+                </Checker.Connect>
+              </div>
+            ) : null}
 
             <DialogContent>
               <DialogHeader>

diff --git a/apps/web/src/ui/pool/ConcentratedPositionsTable/ConcentratedPositionsTable.tsx b/apps/web/src/ui/pool/ConcentratedPositionsTable/ConcentratedPositionsTable.tsx
@@ -128,7 +128,6 @@ export const ConcentratedPositionsTable: FC<ConcentratedPositionsTableProps> =
                   ({_positions.length})
                 </span>
               </span>
-              {actions}
               {!hideCollectAllButton ? (
                 <ConcentratedLiquidityCollectAllWidget
                   positions={_positions}
@@ -164,6 +163,7 @@ export const ConcentratedPositionsTable: FC<ConcentratedPositionsTableProps> =
                   </Button>
                 </LinkInternal>
               ) : null}
+              {actions}
             </div>
           </CardTitle>
         </CardHeader>