feat: allow gasLimit field to be disabled

LedgerHQ · Sep 10, 2024 · 2b26edd · 2b26edd
1 parent 7f53095
commit 2b26edd
Show file tree

Hide file tree

Showing 5 changed files with 23 additions and 12 deletions.
diff --git a/.changeset/swift-turtles-taste.md b/.changeset/swift-turtles-taste.md
@@ -0,0 +1,5 @@
+---
+"ledger-live-desktop": minor
+---
+
+feat: allow gasLimit field to be disabled
diff --git a/apps/ledger-live-desktop/src/renderer/families/evm/SendAmountFields/GasLimitField.tsx b/apps/ledger-live-desktop/src/renderer/families/evm/SendAmountFields/GasLimitField.tsx
@@ -17,6 +17,7 @@ const AdvancedOptions: NonNullable<EvmFamily["sendAmountFields"]>["component"] =
   transaction,
   status,
   updateTransaction,
+  disableEditGasLimit,
 }) => {
   invariant(transaction.family === "evm", "AdvancedOptions: evm family expected");
   invariant(account, "Account required");
@@ -63,18 +64,20 @@ const AdvancedOptions: NonNullable<EvmFamily["sendAmountFields"]>["component"] =
       ) : (
         <Box horizontal justifyContent="left">
           <Label color="p.theme.colors.palette.text.shade100">{gasLimit.toString()}</Label>
-          <Button
-            borderRadius={4}
-            variant="shade"
-            outline
-            size="small"
-            onClick={onEditClick}
-            ml={2}
-          >
-            <Box horizontal alignItems="center">
-              {t("send.steps.details.edit")}
-            </Box>
-          </Button>
+          {!disableEditGasLimit && (
+            <Button
+              borderRadius={4}
+              variant="shade"
+              outline
+              size="small"
+              onClick={onEditClick}
+              ml={2}
+            >
+              <Box horizontal alignItems="center">
+                {t("send.steps.details.edit")}
+              </Box>
+            </Button>
+          )}
         </Box>
       )}
     </Box>

diff --git a/apps/ledger-live-desktop/src/renderer/families/types.ts b/apps/ledger-live-desktop/src/renderer/families/types.ts
@@ -216,6 +216,7 @@ export type LLDCoinFamily<
       bridgePending?: boolean;
       trackProperties?: Record<string, unknown>;
       transactionToUpdate?: T;
+      disableEditGasLimit?: boolean;
     }>;
     fields?: string[];
   };

diff --git a/apps/ledger-live-desktop/src/renderer/modals/Send/SendAmountFields.tsx b/apps/ledger-live-desktop/src/renderer/modals/Send/SendAmountFields.tsx
@@ -15,6 +15,7 @@ export type SendAmountFieldsProps = {
   trackProperties?: Record<string, unknown>;
   transactionToUpdate?: Transaction;
   disableSlowStrategy?: boolean;
+  disableEditGasLimit?: boolean;
 };
 
 const AmountRelatedField = (props: SendAmountFieldsProps) => {

diff --git a/.../ledger-live-desktop/src/renderer/screens/exchange/Swap2/Form/FeesDrawerLiveApp/index.tsx b/.../ledger-live-desktop/src/renderer/screens/exchange/Swap2/Form/FeesDrawerLiveApp/index.tsx
@@ -124,6 +124,7 @@ export default function FeesDrawerLiveApp({
             updateTransaction={handleUpdateTransaction}
             mapStrategies={mapStrategies}
             disableSlowStrategy={disableSlowStrategy}
+            disableEditGasLimit={true}
             trackProperties={{
               page: "Swap quotes",
               ...swapDefaultTrack,