update

Author: li-jia-nan

src/Popup/Arrow.tsx

@@ -4,7 +4,7 @@ import type { AlignType, ArrowPos, ArrowTypeOuter } from '../interface';
 
 export interface ArrowProps {
   prefixCls: string;
-  align: AlignType;
+  align?: AlignType;
   arrow: ArrowTypeOuter;
   arrowPos: ArrowPos;
 }

src/hooks/useOffsetStyle.ts

@@ -28,8 +28,8 @@ export default function useOffsetStyle(
     const { points } = align;
     const dynamicInset =
       align.dynamicInset || (align as any)._experimental?.dynamicInset;
-    const alignRight = dynamicInset && points[0][1] === 'r';
-    const alignBottom = dynamicInset && points[0][0] === 'b';
+    const alignRight = dynamicInset && points?.[0][1] === 'r';
+    const alignBottom = dynamicInset && points?.[0][0] === 'b';
 
     if (alignRight) {
       offsetStyle.right = offsetR;

src/index.tsx

@@ -13,7 +13,8 @@ import {
   useLayoutEffect,
 } from '@rc-component/util';
 import * as React from 'react';
-import Popup, { type MobileConfig } from './Popup';
+import Popup from './Popup';
+import type { MobileConfig } from './Popup';
 import type { TriggerContextProps } from './context';
 import TriggerContext, { UniqueContext } from './context';
 import useAction from './hooks/useAction';
@@ -24,15 +25,14 @@ import useWinClick from './hooks/useWinClick';
 import type { PortalProps } from '@rc-component/portal';
 import { isPointInSafeHoverArea } from './util/safeHover';
 import type { SafeHoverPoint } from './util/safeHover';
-
 import type {
   ActionType,
   AlignType,
   ArrowPos,
   ArrowTypeOuter,
   BuildInPlacements,
 } from './interface';
-import { getAlignPopupClassName } from './util';
+import { clamp, getAlignPopupClassName } from './util';
 
 export type {
   ActionType,
@@ -480,10 +480,8 @@ export function generateTrigger(
             popupEle.getBoundingClientRect(),
           );
 
-        const refreshDelay = Math.max(
-          1000 / 60,
-          Math.min(mouseLeaveDelay * 1000, 1000),
-        );
+        // Between 1 frame and 1 second
+        const refreshDelay = clamp(mouseLeaveDelay * 1000, 1000 / 60, 1000);
 
         const scheduleRefresh = () => {
           const safeHover = safeHoverRef.current;

src/util.ts

@@ -52,8 +52,11 @@ export function collectScroller(ele: HTMLElement) {
 
   while (current) {
     const { overflowX, overflowY, overflow } =
-      getWin(current).getComputedStyle(current);
-    if ([overflowX, overflowY, overflow].some((o) => scrollStyle.includes(o))) {
+      getWin(current)?.getComputedStyle(current) || {};
+
+    if (
+      [overflowX, overflowY, overflow].some((o) => o && scrollStyle.includes(o))
+    ) {
       scrollerList.push(current);
     }
 
@@ -67,10 +70,17 @@ export function toNum(num: number, defaultValue = 1) {
   return Number.isNaN(num) ? defaultValue : num;
 }
 
-function getPxValue(val: string) {
-  return toNum(parseFloat(val), 0);
+function getPxValue(val?: string) {
+  if (!val) {
+    return 0;
+  }
+  return toNum(Number.parseFloat(val), 0);
 }
 
+export const clamp = (num: number, min: number, max: number) => {
+  return Math.min(Math.max(num, min), max);
+};
+
 export interface VisibleArea {
   left: number;
   top: number;
@@ -119,7 +129,7 @@ export function getVisibleArea(
       borderBottomWidth,
       borderLeftWidth,
       borderRightWidth,
-    } = getWin(ele).getComputedStyle(ele);
+    } = getWin(ele)?.getComputedStyle(ele) || {};
 
     const eleRect = ele.getBoundingClientRect();
     const {