From abcd239f25960d50251d351d9587b81741b4c418 Mon Sep 17 00:00:00 2001
From: Haileyesus <something@gmail.com>
Date: Fri, 6 Feb 2026 11:54:00 +0300
Subject: [PATCH] refactor: remove showQuickSettings state and toggle from
 AppContent, manage isOpen state directly in QuickSettingsPanel

---
 src/App.jsx                           | 4 +---
 src/components/QuickSettingsPanel.jsx | 7 +++----
 2 files changed, 4 insertions(+), 7 deletions(-)

diff --git a/src/App.jsx b/src/App.jsx
index ee1959c..ff87d20 100644
--- a/src/App.jsx
+++ b/src/App.jsx
@@ -59,7 +59,7 @@ function AppContent() {
   const [isInputFocused, setIsInputFocused] = useState(false);
   const [showSettings, setShowSettings] = useState(false);
   const [settingsInitialTab, setSettingsInitialTab] = useState('agents');
-  const [showQuickSettings, setShowQuickSettings] = useState(false);
+  
   const { preferences, setPreference } = useUiPreferences();
   const { autoExpandTools, showRawParameters, showThinking, autoScrollToBottom, sendByCtrlEnter, sidebarVisible } = preferences;
   // Session Protection System: Track sessions with active conversations to prevent
@@ -712,8 +712,6 @@ function AppContent() {
       {/* Quick Settings Panel - Only show on chat tab */}
       {activeTab === 'chat' && (
         <QuickSettingsPanel
-          isOpen={showQuickSettings}
-          onToggle={setShowQuickSettings}
           autoExpandTools={autoExpandTools}
           onAutoExpandChange={(value) => setPreference('autoExpandTools', value)}
           showRawParameters={showRawParameters}
diff --git a/src/components/QuickSettingsPanel.jsx b/src/components/QuickSettingsPanel.jsx
index 6da0289..3e0de50 100644
--- a/src/components/QuickSettingsPanel.jsx
+++ b/src/components/QuickSettingsPanel.jsx
@@ -21,8 +21,6 @@ import { useTheme } from '../contexts/ThemeContext';
 import LanguageSelector from './LanguageSelector';
 
 const QuickSettingsPanel = ({
-  isOpen,
-  onToggle,
   autoExpandTools,
   onAutoExpandChange,
   showRawParameters,
@@ -36,7 +34,8 @@ const QuickSettingsPanel = ({
   isMobile
 }) => {
   const { t } = useTranslation('settings');
-  const [localIsOpen, setLocalIsOpen] = useState(isOpen);
+  const [ isOpen, setIsOpen ] = useState(false); 
+  const [localIsOpen, setLocalIsOpen] = useState(false); // ! Is this necessary? Can we just use isOpen?
   const [whisperMode, setWhisperMode] = useState(() => {
     return localStorage.getItem('whisperMode') || 'default';
   });
@@ -208,7 +207,7 @@ const QuickSettingsPanel = ({
 
     const newState = !localIsOpen;
     setLocalIsOpen(newState);
-    onToggle(newState);
+    setIsOpen(newState);
   };
 
   return (