Instant curtain review fixes

gmx-io · Dec 19, 2024 · 2c50cde · 2c50cde
1 parent 681553d
commit 2c50cde
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 8 deletions.
diff --git a/src/components/Footer/Footer.css b/src/components/Footer/Footer.css
@@ -10,10 +10,6 @@
   border-top: 1px solid var(--color-gray-950);
 }
 
-.Footer-wrapper.mobile-trade-page {
-  padding-bottom: 52px;
-}
-
 .Footer-logo {
   margin-bottom: 2.4rem;
   display: flex;

diff --git a/src/components/Footer/Footer.tsx b/src/components/Footer/Footer.tsx
@@ -28,7 +28,7 @@ export default function Footer({ showRedirectModal, redirectPopupTimestamp, isMo
 
   return (
     <>
-      <div className={cx("Footer-wrapper", { home: isHome, "mobile-trade-page": isMobileTradePage })}>
+      <div className={cx("Footer-wrapper", { home: isHome, "pb-52": isMobileTradePage })}>
         <div className="Footer-logo">
           <img src={logoImg} alt="MetaMask" />
         </div>

diff --git a/src/components/Synthetics/TradeBox/Curtain.tsx b/src/components/Synthetics/TradeBox/Curtain.tsx
@@ -33,11 +33,11 @@ export function Curtain({
 
   const headerClick = useCallback(() => {
     setIsOpen(true);
-  }, []);
+  }, [setIsOpen]);
 
   const handleClick = useCallback(() => {
     setIsOpen((prev) => !prev);
-  }, []);
+  }, [setIsOpen]);
 
   return (
     <>

diff --git a/src/components/Synthetics/TradeBox/TradeBoxHeaderTabs.tsx b/src/components/Synthetics/TradeBox/TradeBoxHeaderTabs.tsx
@@ -1,5 +1,4 @@
 import { msg } from "@lingui/macro";
-import cx from "classnames";
 import { useCallback } from "react";
 import { useHistory } from "react-router-dom";