mirror of
https://github.com/siteboon/claudecodeui.git
synced 2026-02-22 16:47:37 +00:00
refactor: remove showQuickSettings state and toggle from AppContent, manage isOpen state directly in QuickSettingsPanel
This commit is contained in:
@@ -59,7 +59,7 @@ function AppContent() {
|
|||||||
const [isInputFocused, setIsInputFocused] = useState(false);
|
const [isInputFocused, setIsInputFocused] = useState(false);
|
||||||
const [showSettings, setShowSettings] = useState(false);
|
const [showSettings, setShowSettings] = useState(false);
|
||||||
const [settingsInitialTab, setSettingsInitialTab] = useState('agents');
|
const [settingsInitialTab, setSettingsInitialTab] = useState('agents');
|
||||||
const [showQuickSettings, setShowQuickSettings] = useState(false);
|
|
||||||
const { preferences, setPreference } = useUiPreferences();
|
const { preferences, setPreference } = useUiPreferences();
|
||||||
const { autoExpandTools, showRawParameters, showThinking, autoScrollToBottom, sendByCtrlEnter, sidebarVisible } = preferences;
|
const { autoExpandTools, showRawParameters, showThinking, autoScrollToBottom, sendByCtrlEnter, sidebarVisible } = preferences;
|
||||||
// Session Protection System: Track sessions with active conversations to prevent
|
// Session Protection System: Track sessions with active conversations to prevent
|
||||||
@@ -712,8 +712,6 @@ function AppContent() {
|
|||||||
{/* Quick Settings Panel - Only show on chat tab */}
|
{/* Quick Settings Panel - Only show on chat tab */}
|
||||||
{activeTab === 'chat' && (
|
{activeTab === 'chat' && (
|
||||||
<QuickSettingsPanel
|
<QuickSettingsPanel
|
||||||
isOpen={showQuickSettings}
|
|
||||||
onToggle={setShowQuickSettings}
|
|
||||||
autoExpandTools={autoExpandTools}
|
autoExpandTools={autoExpandTools}
|
||||||
onAutoExpandChange={(value) => setPreference('autoExpandTools', value)}
|
onAutoExpandChange={(value) => setPreference('autoExpandTools', value)}
|
||||||
showRawParameters={showRawParameters}
|
showRawParameters={showRawParameters}
|
||||||
|
|||||||
@@ -21,8 +21,6 @@ import { useTheme } from '../contexts/ThemeContext';
|
|||||||
import LanguageSelector from './LanguageSelector';
|
import LanguageSelector from './LanguageSelector';
|
||||||
|
|
||||||
const QuickSettingsPanel = ({
|
const QuickSettingsPanel = ({
|
||||||
isOpen,
|
|
||||||
onToggle,
|
|
||||||
autoExpandTools,
|
autoExpandTools,
|
||||||
onAutoExpandChange,
|
onAutoExpandChange,
|
||||||
showRawParameters,
|
showRawParameters,
|
||||||
@@ -36,7 +34,8 @@ const QuickSettingsPanel = ({
|
|||||||
isMobile
|
isMobile
|
||||||
}) => {
|
}) => {
|
||||||
const { t } = useTranslation('settings');
|
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(() => {
|
const [whisperMode, setWhisperMode] = useState(() => {
|
||||||
return localStorage.getItem('whisperMode') || 'default';
|
return localStorage.getItem('whisperMode') || 'default';
|
||||||
});
|
});
|
||||||
@@ -208,7 +207,7 @@ const QuickSettingsPanel = ({
|
|||||||
|
|
||||||
const newState = !localIsOpen;
|
const newState = !localIsOpen;
|
||||||
setLocalIsOpen(newState);
|
setLocalIsOpen(newState);
|
||||||
onToggle(newState);
|
setIsOpen(newState);
|
||||||
};
|
};
|
||||||
|
|
||||||
return (
|
return (
|
||||||
|
|||||||
Reference in New Issue
Block a user