From d4d4b145c74d613011a6a3191da65e57a8b7d7b4 Mon Sep 17 00:00:00 2001 From: Jerald-Golden-B <jerald@hexrfactory.com> Date: Sat, 29 Mar 2025 14:31:02 +0530 Subject: [PATCH] refactor: improve toggle UI logic in Header and Tools components for better state management --- app/src/components/layout/sidebarLeft/Header.tsx | 6 ++++-- app/src/components/ui/Tools.tsx | 6 +++--- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/app/src/components/layout/sidebarLeft/Header.tsx b/app/src/components/layout/sidebarLeft/Header.tsx index c51de20..06706c6 100644 --- a/app/src/components/layout/sidebarLeft/Header.tsx +++ b/app/src/components/layout/sidebarLeft/Header.tsx @@ -22,8 +22,10 @@ const Header: React.FC = () => { <div className={`toggle-sidebar-ui-button ${!toggleUI ? "active" : ""}`} onClick={() => { - if (activeModule !== "market") setToggleUI(!toggleUI); - localStorage.setItem("navBarUi", JSON.stringify(!toggleUI)); + if (activeModule !== "market") { + setToggleUI(!toggleUI); + localStorage.setItem("navBarUi", JSON.stringify(!toggleUI)); + } }} > <ToggleSidebarIcon /> diff --git a/app/src/components/ui/Tools.tsx b/app/src/components/ui/Tools.tsx index fc85909..c86164c 100644 --- a/app/src/components/ui/Tools.tsx +++ b/app/src/components/ui/Tools.tsx @@ -76,13 +76,13 @@ const Tools: React.FC = () => { setDeleteModels(false); setAddAction(null); setToggleView(true); - localStorage.setItem("navBarUi", JSON.stringify(!toggleThreeD)); + // localStorage.setItem("navBarUi", JSON.stringify(!toggleThreeD)); } else { setToggleView(false); } - setActiveSubTool("cursor"); - setActiveTool("cursor"); + setToggleUI(localStorage.getItem('navBarUi') ? localStorage.getItem('navBarUi') === 'true' : true) setToggleThreeD(!toggleThreeD); + setActiveSubTool("cursor"); setActiveTool("cursor"); };