Revert "test and look over updates"

This reverts commit 15f2c46.

Author: dreamwasp

packages/gamut/src/Popover/hooks.ts

@@ -2,43 +2,6 @@ import { useEffect } from 'react';
 
 import { findResizingParent, findScrollingParent } from './utils';
 
-// Helper function to find all scrolling parents
-const findAllScrollingParents = (element: HTMLElement): HTMLElement[] => {
-  const scrollingParents: HTMLElement[] = [];
-  let currentElement = element.parentElement;
-
-  while (currentElement && currentElement !== document.body) {
-    const { overflow, overflowY, overflowX } = getComputedStyle(currentElement);
-    if (
-      [overflow, overflowY, overflowX].some((val) =>
-        ['scroll', 'auto'].includes(val)
-      )
-    ) {
-      scrollingParents.push(currentElement);
-    }
-    currentElement = currentElement.parentElement;
-  }
-
-  // Always include window for document-level scrolling
-  if (
-    scrollingParents.length === 0 ||
-    !scrollingParents.includes(document.documentElement)
-  ) {
-    scrollingParents.push(document.documentElement);
-  }
-
-  return scrollingParents;
-};
-
-// Debounce utility
-const createDebounce = (func: () => void, delay: number) => {
-  let timeoutId: ReturnType<typeof setTimeout>;
-  return () => {
-    clearTimeout(timeoutId);
-    timeoutId = setTimeout(func, delay);
-  };
-};
-
 export const useScrollingParentEffect = (
   targetRef: React.RefObject<
     Pick<HTMLDivElement, 'getBoundingClientRect' | 'contains'>
@@ -49,42 +12,17 @@ export const useScrollingParentEffect = (
     if (!targetRef.current) {
       return;
     }
-
-    const target = targetRef.current as unknown as HTMLElement;
-    const scrollingParents = findAllScrollingParents(target);
-
-    const updatePosition = () => {
+    const scrollingParent = findScrollingParent(
+      targetRef.current as unknown as HTMLElement
+    );
+    if (!scrollingParent?.addEventListener) {
+      return;
+    }
+    const handler = () => {
       setTargetRect(targetRef?.current?.getBoundingClientRect());
     };
-
-    // Debounced version for performance during rapid scrolling
-    const debouncedUpdate = createDebounce(updatePosition, 10);
-
-    // For immediate updates during scroll
-    const immediateUpdate = () => {
-      updatePosition();
-    };
-
-    const cleanup: (() => void)[] = [];
-
-    // Add listeners to all scrolling parents
-    scrollingParents.forEach((parent) => {
-      if (parent.addEventListener) {
-        // Use immediate update for smoother experience
-        parent.addEventListener('scroll', immediateUpdate, { passive: true });
-        cleanup.push(() =>
-          parent.removeEventListener('scroll', immediateUpdate)
-        );
-      }
-    });
-
-    // Also listen to window scroll as a fallback
-    window.addEventListener('scroll', immediateUpdate, { passive: true });
-    cleanup.push(() => window.removeEventListener('scroll', immediateUpdate));
-
-    return () => {
-      cleanup.forEach((fn) => fn());
-    };
+    scrollingParent.addEventListener('scroll', handler);
+    return () => scrollingParent.removeEventListener('scroll', handler);
   }, [targetRef, setTargetRect]);
 };
 

packages/gamut/src/PopoverContainer/PopoverContainer.tsx

@@ -75,32 +75,9 @@ export const PopoverContainer: React.FC<PopoverContainerProps> = ({
     setTargetRect(targetRef?.current?.getBoundingClientRect());
   }, [targetRef, isOpen, winW, winH, winX, winY]);
 
-  // Enhanced scroll handling that updates both targetRect and containers
-  const updateTargetPosition = useCallback(
-    (rect?: DOMRect) => {
-      const target = targetRef?.current;
-      if (!target) return;
-
-      const newRect = rect || target.getBoundingClientRect();
-      setTargetRect(newRect);
-
-      // Get current scroll position for accurate container calculation
-      const currentScrollX =
-        window.pageXOffset || document.documentElement.scrollLeft;
-      const currentScrollY =
-        window.pageYOffset || document.documentElement.scrollTop;
-
-      // Also update containers to ensure positioning stays in sync
-      setContainers(
-        getContainers(target, inline, { x: currentScrollX, y: currentScrollY })
-      );
-    },
-    [targetRef, inline]
-  );
-
-  useScrollingParentEffect(targetRef, updateTargetPosition);
+  useScrollingParentEffect(targetRef, setTargetRect);
 
-  useResizingParentEffect(targetRef, updateTargetPosition);
+  useResizingParentEffect(targetRef, setTargetRect);
 
   useIsomorphicLayoutEffect(() => {
     if (