moves all unmount code inside useEffectEvent

Author: bernhardoj

src/pages/Search/SearchTypeMenuWide.tsx

@@ -63,23 +63,18 @@ function Section({section, hash, activeItemIndex, sectionStartIndex, reportCount
     ]);
 
     const [isExpanded, setIsExpanded] = useState(true);
-    const isExpandedRef = useRef(isExpanded);
 
-    const onCollapsedCallback = useEffectEvent(onCollapsed);
-
-    useEffect(() => {
-        isExpandedRef.current = isExpanded;
-    }, [isExpanded]);
+    const onUnmount = useEffectEvent(() => {
+        if (isExpanded) {
+            return;
+        }
+        // When the section is removed/unmounted while collapsed,
+        // notify the parent that the section is no longer collapsed.
+        onCollapsed(false);
+    });
 
-    // When the section is removed/unmounted while collapsed,
-    // notify the parent that the section is no longer collapsed.
     useEffect(() => {
-        return () => {
-            if (isExpandedRef.current) {
-                return;
-            }
-            onCollapsedCallback(false);
-        };
+        return () => onUnmount();
     }, []);
 
     return (