fix: Internal use of onDismissOnScroll

Author: Who-is-PS

src/popover/container.tsx

@@ -43,6 +43,8 @@ interface PopoverContainerProps {
   allowVerticalOverflow?: boolean;
   // Whether the popover should be hidden when the trigger is scrolled away.
   hideOnOverscroll?: boolean;
+  // Callback fired when the popover detects it has been scrolled away (overscrolled).
+  onOverscroll?: () => void;
   hoverArea?: boolean;
   className?: string;
 }
@@ -64,6 +66,7 @@ export default function PopoverContainer({
   allowScrollToFit,
   allowVerticalOverflow,
   hideOnOverscroll,
+  onOverscroll,
   hoverArea,
   className,
 }: PopoverContainerProps) {
@@ -101,6 +104,15 @@ export default function PopoverContainer({
       minVisibleBlockSize,
     });
 
+  // Call onOverscroll callback when isOverscrolling becomes true
+  const prevIsOverscrolling = useRef(false);
+  useLayoutEffect(() => {
+    if (isOverscrolling && !prevIsOverscrolling.current && onOverscroll) {
+      onOverscroll();
+    }
+    prevIsOverscrolling.current = isOverscrolling;
+  }, [isOverscrolling, onOverscroll]);
+
   // Recalculate position when properties change.
   useLayoutEffect(() => {
     updatePositionHandler();

src/select/parts/item.tsx

@@ -122,6 +122,7 @@ const Item = (
                 content={disabledReason!}
                 position="right"
                 __dismissOnScroll={true}
+                __onDismissOnScroll={() => setCanShowTooltip(false)}
                 onEscape={() => setCanShowTooltip(false)}
               />
             )}

src/tooltip/interfaces.ts

@@ -52,6 +52,12 @@ export interface InternalTooltipProps extends TooltipProps {
    */
   __dismissOnScroll?: boolean;
 
+  /**
+   * Callback fired when the tooltip is dismissed due to scroll.
+   * @internal
+   */
+  __onDismissOnScroll?: NonCancelableEventHandler;
+
   /**
    * Additional CSS class for the tooltip container.
    * @internal

src/tooltip/internal.tsx

@@ -24,12 +24,14 @@ export default function InternalTooltip({
   trackKey,
   position = 'top',
   __dismissOnScroll,
+  __onDismissOnScroll,
   onEscape,
   __internalRootRef,
   ...restProps
 }: InternalTooltipComponentProps) {
   const baseProps = getBaseProps(restProps);
   const trackRef = React.useRef<HTMLElement | SVGElement | null>(null);
+  const [isVisible, setIsVisible] = React.useState(true);
 
   // Update the ref with the current tracked element
   React.useEffect(() => {
@@ -64,6 +66,15 @@ export default function InternalTooltip({
     };
   }, [onEscape]);
 
+  const handleDismissOnScroll = React.useCallback(() => {
+    setIsVisible(false);
+    fireNonCancelableEvent(__onDismissOnScroll);
+  }, [__onDismissOnScroll]);
+
+  if (!isVisible) {
+    return null;
+  }
+
   return (
     <Portal>
       <div
@@ -83,6 +94,7 @@ export default function InternalTooltip({
               zIndex={7000}
               arrow={position => <PopoverArrow position={position} />}
               hideOnOverscroll={__dismissOnScroll}
+              onOverscroll={handleDismissOnScroll}
             >
               <PopoverBody dismissButton={false} dismissAriaLabel={undefined} onDismiss={undefined} header={undefined}>
                 {content}