fix(calendar): prevent redundant range updates (#395)

packages/calendar/src/utils/stateful/calendar-state/__test__/set-range.spec.ts

@@ -52,6 +52,20 @@ describe('calendar state', () => {
         end: '2023-09-13 23:59',
       })
     })
+
+    it('should not update the range if both start and end remain the same', () => {
+      const state = createCalendarState(config, timeUnitsImpl)
+      state.view.value = InternalViewName.Day
+      state.range.value = {
+        start: '2023-09-13 00:00',
+        end: '2023-09-13 23:59',
+      }
+      const originalRange = state.range.value
+
+      state.handleDateSelection('2023-09-13')
+
+      expect(state.range.value).toBe(originalRange) // checking object equality is wanted here
+    })
   })
 
   describe('setting the range in a hybrid day', () => {

packages/calendar/src/utils/stateful/calendar-state/calendar-state.impl.ts

@@ -40,12 +40,19 @@ export const createCalendarState = (
     const selectedView = calendarConfig.views.find(
       (availableView) => availableView.name === view.value
     )
-    range.value = (selectedView as View).setDateRange({
+    const newRange = (selectedView as View).setDateRange({
       calendarConfig,
       date,
       range,
       timeUnitsImpl,
     })
+    if (
+      newRange.start === range.value?.start &&
+      newRange.end === range.value?.end
+    )
+      return
+
+    range.value = newRange
   }
 
   const isCalendarSmall = signal<boolean | undefined>(undefined)