fixup! refactor(DatePicker): Break up, extract handlers and consolida…

…te state
Royal-Navy · Nov 8, 2024 · 5f0ed87 · 5f0ed87
1 parent aa6531c
commit 5f0ed87
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/packages/react-component-library/src/components/DatePicker/types.ts b/packages/react-component-library/src/components/DatePicker/types.ts
@@ -21,7 +21,7 @@ export const DATEPICKER_ACTION = {
 
 interface ResetAction {
   type: typeof DATEPICKER_ACTION.RESET
-  data: Omit<DatePickerState, 'hasError' | 'inputValue'>
+  data: Omit<DatePickerState, 'hasError' | 'inputValue' | 'isOpen'>
 }
 
 interface UpdateAction {

diff --git a/packages/react-component-library/src/components/DatePicker/useDatePickerContext.tsx b/packages/react-component-library/src/components/DatePicker/useDatePickerContext.tsx
@@ -60,7 +60,7 @@ function reducer(
 ): DatePickerState {
   switch (action.type) {
     case DATEPICKER_ACTION.RESET:
-      return init(action.data)
+      return init({ ...action.data, isOpen: state.isOpen })
     case DATEPICKER_ACTION.UPDATE:
       return { ...state, ...action.data }
     case DATEPICKER_ACTION.REFRESH_HAS_ERROR:
@@ -154,7 +154,6 @@ export function DatePickerProvider({
           endDate: endDate ?? null,
           currentMonth,
           datePickerFormat,
-          isOpen: initialIsOpen,
           calendarTableVariant: CALENDAR_TABLE_VARIANT.HIDE,
         },
       })