From 6cd2c276ebc829ffd05a4719881308de22d0f8ee Mon Sep 17 00:00:00 2001
From: TinsFox <fox@tinsfox.com>
Date: Sun, 3 Nov 2024 00:35:14 +0800
Subject: [PATCH] fix: fix duplicate dependency issues

---
 src/components/nav-sidebar/nav-main.tsx | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/src/components/nav-sidebar/nav-main.tsx b/src/components/nav-sidebar/nav-main.tsx
index 48cea86..928cfe7 100644
--- a/src/components/nav-sidebar/nav-main.tsx
+++ b/src/components/nav-sidebar/nav-main.tsx
@@ -23,6 +23,8 @@ import {
 } from "@/components/ui/sidebar"
 import type { IMenu } from "@/models/menu"
 
+import { Button } from "../ui/button"
+
 export function NavMain({
   items,
 }: {
@@ -59,7 +61,7 @@ export function NavMain({
     if (hasChanges) {
       setOpenItems(newOpenItems)
     }
-  }, [location.pathname, items, isPathActive, isParentActive, setOpenItems, openItems])
+  }, [location.pathname, items, isPathActive, isParentActive])
 
   const handleToggle = (title: string) => {
     setOpenItems((prev) => ({
@@ -83,8 +85,10 @@ export function NavMain({
     <SidebarGroup>
       <div className="flex items-center justify-between px-2">
         <SidebarGroupLabel>Platform</SidebarGroupLabel>
-        <button
+        <Button
           type="button"
+          size="icon"
+          variant="ghost"
           onClick={handleToggleAll}
           className="rounded-md p-1 hover:bg-muted"
           title={items.every((item) => openItems[item.title]) ? "Collapse all" : "Expand all"}
@@ -94,7 +98,7 @@ export function NavMain({
           ) : (
             <Maximize2 className="size-4" />
           )}
-        </button>
+        </Button>
       </div>
       <SidebarMenu>
         {items.map((item) => (