From 4693deed7fbebebd0cb1f2c81e3eeaba33b3ce96 Mon Sep 17 00:00:00 2001 From: Vishnu Date: Tue, 13 May 2025 12:43:54 +0530 Subject: [PATCH] Refactor handleSidebarShortcuts to improve module condition check and streamline localStorage updates for UI state management --- .../utils/shortcutkeys/handleShortcutKeys.ts | 45 ++++++++++++------- 1 file changed, 29 insertions(+), 16 deletions(-) diff --git a/app/src/utils/shortcutkeys/handleShortcutKeys.ts b/app/src/utils/shortcutkeys/handleShortcutKeys.ts index c3346b8..c8d735a 100644 --- a/app/src/utils/shortcutkeys/handleShortcutKeys.ts +++ b/app/src/utils/shortcutkeys/handleShortcutKeys.ts @@ -108,26 +108,39 @@ const KeyPressListener: React.FC = () => { const handleSidebarShortcuts = (key: string) => { - if (activeModule !== "market") { - if (key === "Ctrl+\\") { + if (activeModule === "market") return; + + const updateLocalStorage = (left: boolean, right: boolean) => { + localStorage.setItem("navBarUiLeft", JSON.stringify(left)); + localStorage.setItem("navBarUiRight", JSON.stringify(right)); + }; + + switch (key) { + case "Ctrl+\\": if (toggleUILeft === toggleUIRight) { - setToggleUI(!toggleUILeft, !toggleUIRight); - } - else { + const newState = !toggleUILeft; + setToggleUI(newState, newState); + updateLocalStorage(newState, newState); + } else { setToggleUI(true, true); + updateLocalStorage(true, true); } - return; - } - if (key === "Ctrl+]") { - setToggleUI(toggleUILeft, !toggleUIRight); - return; - } - if (key === "Ctrl+[") { - setToggleUI(!toggleUILeft, toggleUIRight); - return; - } + break; + + case "Ctrl+]": + setToggleUI(toggleUILeft, !toggleUIRight); + updateLocalStorage(toggleUILeft, !toggleUIRight); + break; + + case "Ctrl+[": + setToggleUI(!toggleUILeft, toggleUIRight); + updateLocalStorage(!toggleUILeft, toggleUIRight); + break; + + default: + break; } - } + }; const handleKeyPress = (event: KeyboardEvent) => {