fix: avoid setting dateInView when value stays the same (schummar#103)

* fix: avoid setting dateInView when value stays the same * inline comparison --------- Co-authored-by: Michael Förg <[email protected]>
fer0n · Nov 28, 2023 · d070acf · d070acf
1 parent e8470b5
commit d070acf
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/src/components/datePicker.tsx b/src/components/datePicker.tsx
@@ -203,6 +203,16 @@ export function dateClamp(date: Date, min?: Date, max?: Date) {
   return date;
 }
 
+function getValueForComparison(value?: Date | DateRange | null) {
+  if (!value) {
+    return value;
+  }
+  if (value instanceof Date) {
+    return value.getTime();
+  }
+  return `${value.min?.getTime()}-${value.max?.getTime()}`;
+}
+
 export function DatePicker(props: DatePickerProps) {
   const context = useContext(DatePickerContext);
 
@@ -366,7 +376,7 @@ export function DatePicker(props: DatePickerProps) {
       setDateInView(
         value === null ? defaultDateInView ?? mountTime : value instanceof Date ? value : value.max,
       ),
-    [value, defaultDateInView, mountTime],
+    [getValueForComparison(value), getValueForComparison(defaultDateInView), mountTime],
   );
 
   useEffect(() => {