fix:adjust hasSubmitValue check for submitIndexRef

react-component · Dec 19, 2024 · caaebf8 · caaebf8
1 parent 0708b14
commit caaebf8
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 8 deletions.
diff --git a/src/PickerInput/RangePicker.tsx b/src/PickerInput/RangePicker.tsx
@@ -260,6 +260,7 @@ function RangePicker<DateType extends object = any>(
     setActiveIndex,
     nextActiveIndex,
     activeIndexList,
+    submitIndexRef,
   ] = useRangeActive(disabled, allowEmpty, mergedOpen);
 
   const onSharedFocus = (event: React.FocusEvent<HTMLElement>, index?: number) => {
@@ -413,7 +414,7 @@ function RangePicker<DateType extends object = any>(
     if (date) {
       nextValue = fillCalendarValue(date, activeIndex);
     }
-
+    submitIndexRef.current = activeIndex;
     // Get next focus index
     const nextIndex = nextActiveIndex(nextValue);
 
@@ -641,7 +642,7 @@ function RangePicker<DateType extends object = any>(
       needConfirm &&
       // Not change index if is not filled
       !allowEmpty[lastActiveIndex] &&
-      !hasSubmitValue(lastActiveIndex) &&
+      !hasSubmitValue(lastActiveIndex, submitIndexRef.current) &&
       calendarValue[lastActiveIndex]
     ) {
       selectorRef.current.focus({ index: lastActiveIndex });

diff --git a/src/PickerInput/hooks/useRangeActive.ts b/src/PickerInput/hooks/useRangeActive.ts
@@ -23,12 +23,15 @@ export default function useRangeActive<DateType>(
   setActiveIndex: (index: number) => void,
   nextActiveIndex: NextActive<DateType>,
   activeList: number[],
+  submitIndexRef: React.MutableRefObject<number | null>,
 ] {
   const [activeIndex, setActiveIndex] = React.useState(0);
   const [focused, setFocused] = React.useState<boolean>(false);
 
   const activeListRef = React.useRef<number[]>([]);
 
+  const submitIndexRef = React.useRef<number | null>(null);
+
   const lastOperationRef = React.useRef<OperationType>(null);
 
   const triggerFocus = (nextFocus: boolean) => {
@@ -62,6 +65,7 @@ export default function useRangeActive<DateType>(
   useLockEffect(focused || mergedOpen, () => {
     if (!focused) {
       activeListRef.current = [];
+      submitIndexRef.current = null;
     }
   });
 
@@ -79,5 +83,6 @@ export default function useRangeActive<DateType>(
     setActiveIndex,
     nextActiveIndex,
     activeListRef.current,
+    submitIndexRef,
   ];
 }
diff --git a/src/PickerInput/hooks/useRangeValue.ts b/src/PickerInput/hooks/useRangeValue.ts
@@ -186,7 +186,7 @@ export default function useRangeValue<ValueType extends DateType[], DateType ext
   /** Trigger `onChange` directly without check `disabledDate` */
   triggerSubmitChange: (value: ValueType) => boolean,
   /** Tell `index` has filled value in it */
-  hasSubmitValue: (index: number) => boolean,
+  hasSubmitValue: (index: number, submitIndex: number | null) => boolean,
 ] {
   const {
     // MISC
@@ -333,11 +333,8 @@ export default function useRangeValue<ValueType extends DateType[], DateType ext
   );
 
   // ============================ Check =============================
-  function hasSubmitValue(index: number) {
-    return (
-      !!submitValue()[index] &&
-      isSame(generateConfig, locale, submitValue()[index], getCalendarValue()[index], picker)
-    );
+  function hasSubmitValue(index: number, submitIndex: number | null) {
+    return submitIndex === index;
   }
 
   // ============================ Return ============================