chore: adjust open order (#1169)

Author: zombieJ

src/BaseSelect/index.tsx

@@ -570,7 +570,9 @@ const BaseSelect = React.forwardRef<BaseSelectRef, BaseSelectProps>((props, ref)
     }
 
     if (!disabled) {
-      triggerOpen(false);
+      triggerOpen(false, {
+        lazy: true,
+      });
       onBlur?.(event);
     }
   };
@@ -582,7 +584,9 @@ const BaseSelect = React.forwardRef<BaseSelectRef, BaseSelectProps>((props, ref)
     // We should give focus back to selector if clicked item is not focusable
     if (popupElement?.contains(target as HTMLElement) && triggerOpen) {
       // Tell `open` not to close since it's safe in the popup
-      triggerOpen(true, true);
+      triggerOpen(true, {
+        ignoreNext: true,
+      });
     }
 
     onMouseDown?.(event, ...restArgs);

src/hooks/useOpen.ts

@@ -22,7 +22,13 @@ const macroTask = (fn: VoidFunction, times = 1) => {
  * Trigger by latest open call, if nextOpen is undefined, means toggle.
  * ignoreNext will skip next call in the macro task queue.
  */
-export type TriggerOpenType = (nextOpen?: boolean, ignoreNext?: boolean) => void;
+export type TriggerOpenType = (
+  nextOpen?: boolean,
+  config?: {
+    ignoreNext?: boolean;
+    lazy?: boolean;
+  },
+) => void;
 
 /**
  * When `open` is controlled, follow the controlled value;
@@ -61,30 +67,34 @@ export default function useOpen(
     internalSetOpen(nextOpen);
   });
 
-  const toggleOpen = useEvent<TriggerOpenType>((nextOpen?: boolean, ignoreNext = false) => {
+  const toggleOpen = useEvent<TriggerOpenType>((nextOpen, config = {}) => {
+    const { ignoreNext = false, lazy = false } = config;
+
     taskIdRef.current += 1;
     const id = taskIdRef.current;
 
     const nextOpenVal = typeof nextOpen === 'boolean' ? nextOpen : !mergedOpen;
 
     // Since `mergedOpen` is post-processed, we need to check if the value really changed
-    if (nextOpenVal) {
-      triggerEvent(true);
-
-      // Lock if needed
-      if (ignoreNext) {
-        taskLockRef.current = ignoreNext;
-
-        macroTask(() => {
-          taskLockRef.current = false;
-        }, 2);
+    if (nextOpenVal || !lazy) {
+      if (!taskLockRef.current) {
+        triggerEvent(nextOpenVal);
+
+        // Lock if needed
+        if (ignoreNext) {
+          taskLockRef.current = ignoreNext;
+
+          macroTask(() => {
+            taskLockRef.current = false;
+          }, 2);
+        }
       }
       return;
     }
 
     macroTask(() => {
       if (id === taskIdRef.current && !taskLockRef.current) {
-        triggerEvent(false);
+        triggerEvent(nextOpenVal);
       }
     });
   });