fix: RangePicker double click should only act on needConfirm case (#760)

* fix: double click should not effect on simply picker

* test: add test case

* chore: clean up

Author: zombieJ

src/PickerInput/Popup/PopupPanel.tsx

@@ -21,8 +21,16 @@ export type PopupPanelProps<DateType extends object = any> = MustProp<DateType>
 export default function PopupPanel<DateType extends object = any>(
   props: PopupPanelProps<DateType>,
 ) {
-  const { picker, multiplePanel, pickerValue, onPickerValueChange, onSubmit, range, hoverValue } =
-    props;
+  const {
+    picker,
+    multiplePanel,
+    pickerValue,
+    onPickerValueChange,
+    needConfirm,
+    onSubmit,
+    range,
+    hoverValue,
+  } = props;
   const { prefixCls, generateConfig } = React.useContext(PickerContext);
 
   // ======================== Offset ========================
@@ -46,7 +54,9 @@ export default function PopupPanel<DateType extends object = any>(
   // ======================= Context ========================
   const sharedContext: PickerHackContextProps = {
     onCellDblClick: () => {
-      onSubmit();
+      if (needConfirm) {
+        onSubmit();
+      }
     },
   };
 

src/PickerInput/hooks/useFilledProps.ts

@@ -211,5 +211,12 @@ export default function useFilledProps<
     [filledProps, mergedNeedConfirm, mergedInputReadOnly, disabledBoundaryDate],
   );
 
-  return [mergedProps, internalPicker, complexPicker, formatList, maskFormat, isInvalidateDate];
+  return [
+    mergedProps,
+    internalPicker,
+    complexPicker,
+    formatList,
+    maskFormat,
+    isInvalidateDate,
+  ];
 }

tests/new-range.spec.tsx

@@ -732,6 +732,19 @@ describe('NewPicker.Range', () => {
         '1990-09-05 11:00:00',
       ]);
     });
+
+    it('double click should not take action if !needConfirm', () => {
+      const { container } = render(<DayRangePicker />);
+
+      openPicker(container);
+
+      fireEvent.click(findCell(5));
+      fireEvent.click(findCell(5));
+      fireEvent.doubleClick(findCell(5));
+
+      expect(container.querySelectorAll('input')[0]).toHaveValue('1990-09-05');
+      expect(container.querySelectorAll('input')[1]).toHaveValue('1990-09-05');
+    });
   });
 
   describe('open', () => {