From c6e574d08063731a3cc37d7c2abeacd704d562a0 Mon Sep 17 00:00:00 2001 From: kangfenmao Date: Thu, 29 May 2025 13:54:58 +0800 Subject: [PATCH] refactor(Chat, Messages): simplify maxWidth calculations and remove unused showAssistants variable --- src/renderer/src/pages/home/Chat.tsx | 9 ++++----- src/renderer/src/pages/home/Messages/Messages.tsx | 13 +------------ 2 files changed, 5 insertions(+), 17 deletions(-) diff --git a/src/renderer/src/pages/home/Chat.tsx b/src/renderer/src/pages/home/Chat.tsx index dbb10eb1e6..f77c01f590 100644 --- a/src/renderer/src/pages/home/Chat.tsx +++ b/src/renderer/src/pages/home/Chat.tsx @@ -26,7 +26,7 @@ interface Props { const Chat: FC = (props) => { const { assistant } = useAssistant(props.assistant.id) - const { topicPosition, messageStyle, showAssistants } = useSettings() + const { topicPosition, messageStyle } = useSettings() const { showTopics } = useShowTopics() const { isMultiSelectMode } = useChatContext(props.activeTopic) @@ -36,10 +36,9 @@ const Chat: FC = (props) => { const maxWidth = useMemo(() => { const showRightTopics = showTopics && topicPosition === 'right' - const minusAssistantsWidth = showAssistants ? `- var(--assistants-width) - var(--scrollbar-width)` : '' - const minusRightTopicsWidth = showRightTopics ? `- var(--assistants-width) - var(--scrollbar-width)` : '' - return `calc(100vw - var(--sidebar-width) ${minusAssistantsWidth} ${minusRightTopicsWidth})` - }, [showAssistants, showTopics, topicPosition]) + const minusRightTopicsWidth = showRightTopics ? `- var(--assistants-width)` : '' + return `calc(100vw - var(--sidebar-width) - var(--assistants-width) ${minusRightTopicsWidth})` + }, [showTopics, topicPosition]) useHotkeys('esc', () => { contentSearchRef.current?.disable() diff --git a/src/renderer/src/pages/home/Messages/Messages.tsx b/src/renderer/src/pages/home/Messages/Messages.tsx index 430060ba6d..4933ae2190 100644 --- a/src/renderer/src/pages/home/Messages/Messages.tsx +++ b/src/renderer/src/pages/home/Messages/Messages.tsx @@ -86,13 +86,6 @@ const Messages: React.FC = ({ assistant, topic, setActiveTopic, o setHasMore(messages.length > displayCount) }, [messages, displayCount]) - const maxWidth = useMemo(() => { - const showRightTopics = showTopics && topicPosition === 'right' - const minusAssistantsWidth = showAssistants ? `- var(--assistants-width) - var(--scrollbar-width)` : '' - const minusRightTopicsWidth = showRightTopics ? `- var(--assistants-width) - var(--scrollbar-width)` : '' - return `calc(100vw - var(--sidebar-width) ${minusAssistantsWidth} ${minusRightTopicsWidth})` - }, [showAssistants, showTopics, topicPosition]) - const scrollToBottom = useCallback(() => { if (scrollContainerRef.current) { requestAnimationFrame(() => { @@ -277,11 +270,7 @@ const Messages: React.FC = ({ assistant, topic, setActiveTopic, o