feat: reset dialog state on route change and clean up unused click ha…

…ndlers in cart component
franco14lorenzo · Nov 17, 2024 · 401cdec · 401cdec
1 parent bdc0fb6
commit 401cdec
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 6 deletions.
diff --git a/src/app/(store)/components/layout/header/cart/index.tsx b/src/app/(store)/components/layout/header/cart/index.tsx
@@ -106,7 +106,6 @@ const Cart = () => {
                     <Link
                       className="mt-4 h-10 max-w-fit rounded-full border border-white bg-black px-4 py-2 font-medium text-white hover:bg-opacity-80"
                       href="/degustaciones"
-                      onClick={() => setDialogOpen(null)}
                     >
                       Descubre nuestras catas
                       <ArrowRight className="ml-2 inline size-5" />
@@ -143,11 +142,6 @@ const Cart = () => {
                       items.length === 0 &&
                       'pointer-events-none cursor-not-allowed opacity-50'
                     }`}
-                    onClick={() => {
-                      if (items.length !== 0) {
-                        setDialogOpen(null)
-                      }
-                    }}
                   >
                     Finalizar compra
                   </Link>

diff --git a/src/app/contexts/dialogs.tsx b/src/app/contexts/dialogs.tsx
@@ -5,9 +5,12 @@ import React, {
   Dispatch,
   SetStateAction,
   useContext,
+  useEffect,
   useState
 } from 'react'
 
+import { usePathname } from 'next/navigation'
+
 export enum Dialogs {
   Menu,
   Cart
@@ -23,6 +26,15 @@ const DialogContext = createContext<DialogContextType | undefined>(undefined)
 export function DialogProvider({ children }: { children: React.ReactNode }) {
   const [dialogOpen, setDialogOpen] = useState<Dialogs | null>(null)
 
+  const pathname = usePathname()
+  useEffect(() => {
+    setDialogOpen(null)
+
+    return () => {
+      setDialogOpen(null)
+    }
+  }, [pathname])
+
   return (
     <DialogContext.Provider value={[dialogOpen, setDialogOpen]}>
       {children}