Merge pull request #261 from mrgnlabs/j/fix-close-action

fix: close button disabled bug
mrgnlabs · Oct 4, 2023 · 6c46f6a · 6c46f6a · vercel · Oct 6, 2023
2 parents 81fbb89 + 4b5a63e
commit 6c46f6a
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/apps/marginfi-v2-ui/src/components/AssetsList/AssetRow/AssetRow.tsx b/apps/marginfi-v2-ui/src/components/AssetsList/AssetRow/AssetRow.tsx
@@ -675,8 +675,10 @@ const AssetRow: FC<{
               onClick={currentAction === "Connect" ? openWalletSelector : isDust ? closeBalance : borrowOrLend}
               disabled={
                 currentAction !== "Connect" &&
-                ((isDust && uiToNative(bank.userInfo.tokenAccount.balance, bank.info.state.mintDecimals).isZero()) ||
-                  maxAmount === 0)
+                ((isDust &&
+                  uiToNative(bank.userInfo.tokenAccount.balance, bank.info.state.mintDecimals).isZero() &&
+                  currentAction == ActionType.Borrow) ||
+                  (!isDust && maxAmount === 0))
               }
             >
               {isDust ? "Close" : currentAction}

diff --git a/apps/marginfi-v2-ui/src/components/UserPositions/UserPositionRow/UserPositionRow.tsx b/apps/marginfi-v2-ui/src/components/UserPositions/UserPositionRow/UserPositionRow.tsx
@@ -189,8 +189,9 @@ const UserPositionRow: FC<UserPositionRowProps> = ({ activeBankInfo, marginfiAcc
                 uiToNative(
                   activeBankInfo.userInfo.tokenAccount.balance,
                   activeBankInfo.info.state.mintDecimals
-                ).isZero()) ||
-              maxAmount === 0
+                ).isZero() &&
+                !activeBankInfo.position.isLending) ||
+              (!isDust && maxAmount === 0)
             }
           >
             {isDust ? "Close" : activeBankInfo.position.isLending ? "Withdraw" : "Repay"}