From d3d7ea44d09fca4fee72bf0e71b05eba4d979d6e Mon Sep 17 00:00:00 2001 From: Saoud Rizwan <7799382+saoudrizwan@users.noreply.github.com> Date: Wed, 16 Oct 2024 10:45:49 -0400 Subject: [PATCH] Fix bug where switching tasks would not reset expanded rows --- webview-ui/src/components/chat/ChatView.tsx | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/webview-ui/src/components/chat/ChatView.tsx b/webview-ui/src/components/chat/ChatView.tsx index 1d50f00..9bc0c9f 100644 --- a/webview-ui/src/components/chat/ChatView.tsx +++ b/webview-ui/src/components/chat/ChatView.tsx @@ -190,6 +190,10 @@ const ChatView = ({ isHidden, showAnnouncement, hideAnnouncement, showHistoryVie } }, [messages.length]) + useEffect(() => { + setExpandedRows({}) + }, [task?.ts]) + const isStreaming = useMemo(() => { const isLastAsk = !!modifiedMessages.at(-1)?.ask // checking clineAsk isn't enough since messages effect may be called again for a tool for example, set clineAsk to its value, and if the next message is not an ask then it doesn't reset. This is likely due to how much more often we're updating messages as compared to before, and should be resolved with optimizations as it's likely a rendering bug. but as a final guard for now, the cancel button will show if the last message is not an ask const isToolCurrentlyAsking =