From 795e1248aac126f215f288176a981ab7bc299d81 Mon Sep 17 00:00:00 2001 From: iliya Date: Fri, 6 Mar 2026 17:24:52 +0200 Subject: [PATCH] fix: replace ResizeObserver with useEffect for auto-scroll in LeadThoughtsGroup ResizeObserver stops firing once scroll container reaches maxHeight, breaking auto-scroll for long-running live thought groups. Use useEffect on chronologicalThoughts dependency instead. Co-Authored-By: Claude Opus 4.6 --- .../components/team/activity/LeadThoughtsGroup.tsx | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/src/renderer/components/team/activity/LeadThoughtsGroup.tsx b/src/renderer/components/team/activity/LeadThoughtsGroup.tsx index 67f41d31..9e247eed 100644 --- a/src/renderer/components/team/activity/LeadThoughtsGroup.tsx +++ b/src/renderer/components/team/activity/LeadThoughtsGroup.tsx @@ -258,17 +258,12 @@ export const LeadThoughtsGroupRow = ({ return () => observer.disconnect(); }, [onVisible, thoughts]); - // Auto-scroll via ResizeObserver — fires after CSS animations expand content + // Auto-scroll when new thoughts arrive useEffect(() => { const el = scrollRef.current; - if (!el) return; - const observer = new ResizeObserver(() => { - if (isUserScrolledUpRef.current) return; - el.scrollTop = el.scrollHeight; - }); - observer.observe(el); - return () => observer.disconnect(); - }, []); + if (!el || isUserScrolledUpRef.current) return; + el.scrollTop = el.scrollHeight; + }, [chronologicalThoughts]); const handleScroll = useCallback(() => { const el = scrollRef.current;