only onChange on Apply for range mode

medusajs · Oct 6, 2023 · 832edd4 · 832edd4
1 parent 6b4b93b
commit 832edd4
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/packages/ui/src/components/date-picker/date-picker.tsx b/packages/ui/src/components/date-picker/date-picker.tsx
@@ -548,7 +548,6 @@ const RangeDatePicker = ({
     }
 
     setRange(newRange)
-    onChange?.(newRange)
   }
 
   const onCancel = () => {
@@ -641,6 +640,11 @@ const RangeDatePicker = ({
     }`
   }, [range, showTimePicker])
 
+  const onApply = () => {
+    setOpen(false)
+    onChange?.(range)
+  }
+
   return (
     <Primitives.Root open={open} onOpenChange={onOpenChange}>
       <Display
@@ -717,11 +721,7 @@ const RangeDatePicker = ({
                   <Button variant="secondary" type="button" onClick={onCancel}>
                     Cancel
                   </Button>
-                  <Button
-                    variant="primary"
-                    type="button"
-                    onClick={() => setOpen(false)}
-                  >
+                  <Button variant="primary" type="button" onClick={onApply}>
                     Apply
                   </Button>
                 </div>