fix: make isSelected reactive in datepicker (#1466)

themesberg · Oct 27, 2024 · 153a9c9 · 153a9c9
1 parent a88db49
commit 153a9c9
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/src/lib/datepicker/Datepicker.svelte b/src/lib/datepicker/Datepicker.svelte
@@ -161,12 +161,17 @@
     return date.toLocaleDateString(locale, dateFormat);
   }
 
-  function isSelected(day: Date): boolean {
+  function isSameDate(date1: Date | null, date2: Date | null): boolean {
+    if (!date1 || !date2) return false;
+    return date1.toDateString() === date2.toDateString();
+  }
+
+  $: isSelected = (day: Date): boolean => {
     if (range) {
-      return !!(rangeFrom && day.toDateString() === rangeFrom.toDateString()) || !!(rangeTo && day.toDateString() === rangeTo.toDateString());
+      return isSameDate(day, rangeFrom) || isSameDate(day, rangeTo);
     }
-    return !!(value && day.toDateString() === value.toDateString());
-  }
+    return isSameDate(day, value);
+  };
 
   function isInRange(day: Date): boolean {
     if (!range || !rangeFrom || !rangeTo) return false;