@rc-component/trigger 3.8.0 → 3.8.2

package/es/hooks/useAlign.js

@@ -480,12 +480,12 @@ export default function useAlign(open, popupEle, target, placement, builtinPlace
       let offsetX4Right = popupMirrorRect.right - popupRect.x - (nextOffsetX + popupRect.width);
       let offsetY4Bottom = popupMirrorRect.bottom - popupRect.y - (nextOffsetY + popupRect.height);
       if (scaleX === 1) {
-        nextOffsetX = Math.round(nextOffsetX);
-        offsetX4Right = Math.round(offsetX4Right);
+        nextOffsetX = Math.floor(nextOffsetX);
+        offsetX4Right = Math.floor(offsetX4Right);
       }
       if (scaleY === 1) {
-        nextOffsetY = Math.round(nextOffsetY);
-        offsetY4Bottom = Math.round(offsetY4Bottom);
+        nextOffsetY = Math.floor(nextOffsetY);
+        offsetY4Bottom = Math.floor(offsetY4Bottom);
       }
       const nextOffsetInfo = {
         ready: true,

package/es/index.js

@@ -17,6 +17,7 @@ import useWinClick from "./hooks/useWinClick";
 import { getAlignPopupClassName } from "./util";
 import UniqueProvider from "./UniqueProvider";
 import { useControlledState } from '@rc-component/util';
+import { flushSync } from 'react-dom';
 export { UniqueProvider };
 
 // Removed Props List
@@ -193,19 +194,14 @@ export function generateTrigger(PortalComponent = Portal) {
     }, [mergedOpen, targetEle]);
     const openRef = React.useRef(mergedOpen);
     openRef.current = mergedOpen;
-    const lastTriggerRef = React.useRef([]);
-    lastTriggerRef.current = [];
     const internalTriggerOpen = useEvent(nextOpen => {
-      setInternalOpen(nextOpen);
-
-      // Enter or Pointer will both trigger open state change
-      // We only need take one to avoid duplicated change event trigger
-      // Use `lastTriggerRef` to record last open type
-      if ((lastTriggerRef.current[lastTriggerRef.current.length - 1] ?? mergedOpen) !== nextOpen) {
-        lastTriggerRef.current.push(nextOpen);
-        onOpenChange?.(nextOpen);
-        onPopupVisibleChange?.(nextOpen);
-      }
+      flushSync(() => {
+        if (mergedOpen !== nextOpen) {
+          setInternalOpen(nextOpen);
+          onOpenChange?.(nextOpen);
+          onPopupVisibleChange?.(nextOpen);
+        }
+      });
     });
 
     // Trigger for delay

package/lib/hooks/useAlign.js

@@ -489,12 +489,12 @@ function useAlign(open, popupEle, target, placement, builtinPlacements, popupAli
       let offsetX4Right = popupMirrorRect.right - popupRect.x - (nextOffsetX + popupRect.width);
       let offsetY4Bottom = popupMirrorRect.bottom - popupRect.y - (nextOffsetY + popupRect.height);
       if (scaleX === 1) {
-        nextOffsetX = Math.round(nextOffsetX);
-        offsetX4Right = Math.round(offsetX4Right);
+        nextOffsetX = Math.floor(nextOffsetX);
+        offsetX4Right = Math.floor(offsetX4Right);
       }
       if (scaleY === 1) {
-        nextOffsetY = Math.round(nextOffsetY);
-        offsetY4Bottom = Math.round(offsetY4Bottom);
+        nextOffsetY = Math.floor(nextOffsetY);
+        offsetY4Bottom = Math.floor(offsetY4Bottom);
       }
       const nextOffsetInfo = {
         ready: true,

package/lib/index.js

@@ -30,6 +30,7 @@ var _useWinClick = _interopRequireDefault(require("./hooks/useWinClick"));
 var _util = require("./util");
 var _UniqueProvider = _interopRequireDefault(require("./UniqueProvider"));
 var _util2 = require("@rc-component/util");
+var _reactDom = require("react-dom");
 function _getRequireWildcardCache(e) { if ("function" != typeof WeakMap) return null; var r = new WeakMap(), t = new WeakMap(); return (_getRequireWildcardCache = function (e) { return e ? t : r; })(e); }
 function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != typeof e && "function" != typeof e) return { default: e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && Object.prototype.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n.default = e, t && t.set(e, n), n; }
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
@@ -207,19 +208,14 @@ function generateTrigger(PortalComponent = _portal.default) {
     }, [mergedOpen, targetEle]);
     const openRef = React.useRef(mergedOpen);
     openRef.current = mergedOpen;
-    const lastTriggerRef = React.useRef([]);
-    lastTriggerRef.current = [];
     const internalTriggerOpen = (0, _useEvent.default)(nextOpen => {
-      setInternalOpen(nextOpen);
-
-      // Enter or Pointer will both trigger open state change
-      // We only need take one to avoid duplicated change event trigger
-      // Use `lastTriggerRef` to record last open type
-      if ((lastTriggerRef.current[lastTriggerRef.current.length - 1] ?? mergedOpen) !== nextOpen) {
-        lastTriggerRef.current.push(nextOpen);
-        onOpenChange?.(nextOpen);
-        onPopupVisibleChange?.(nextOpen);
-      }
+      (0, _reactDom.flushSync)(() => {
+        if (mergedOpen !== nextOpen) {
+          setInternalOpen(nextOpen);
+          onOpenChange?.(nextOpen);
+          onPopupVisibleChange?.(nextOpen);
+        }
+      });
     });
 
     // Trigger for delay

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rc-component/trigger",
-  "version": "3.8.0",
+  "version": "3.8.2",
   "description": "base abstract trigger component for react",
   "engines": {
     "node": ">=8.x"