From 832edd4428fe00e5a19185ed154f440dbe5f0096 Mon Sep 17 00:00:00 2001
From: Kasper <kasper@medusajs.com>
Date: Fri, 6 Oct 2023 15:57:54 +0200
Subject: [PATCH] only onChange on Apply for range mode

---
 .../ui/src/components/date-picker/date-picker.tsx    | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/packages/ui/src/components/date-picker/date-picker.tsx b/packages/ui/src/components/date-picker/date-picker.tsx
index 0f4cdfe..3832576 100644
--- 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>