Skip to content
This repository was archived by the owner on May 13, 2025. It is now read-only.

Commit fd76cd4

Browse files
authored
fix: Time picker bug fix (#390)
1 parent 910fd4f commit fd76cd4

File tree

1 file changed

+9
-5
lines changed

1 file changed

+9
-5
lines changed

src/components/Header/TimeRange.tsx

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -229,10 +229,10 @@ const CustomTimeRange: FC<CustomTimeRangeProps> = ({ setOpened, resetToRelative
229229
const onTimeSelect = (key: keyof typeof localSelectedRange, time: string) => {
230230
setLocalSelectedRange((state) => {
231231
const [hours, minutes] = time.split(':').map(Number);
232-
const date = state[key];
233-
date.setHours(hours, minutes, 0, 0);
234-
state[key] = date;
235-
return { ...state };
232+
if (isNaN(hours) || isNaN(minutes)) return state;
233+
const updatedDate = new Date(state[key]);
234+
updatedDate.setHours(hours, minutes, 0, 0);
235+
return { ...state, [key]: updatedDate };
236236
});
237237
};
238238

@@ -304,7 +304,11 @@ const CustomTimeRange: FC<CustomTimeRangeProps> = ({ setOpened, resetToRelative
304304
}}
305305
renderDay={(date) => highlightDate(date, 'startTime')}
306306
/>
307-
<TimeInput value={startingTime} onChange={(e) => onTimeSelect('startTime', e.currentTarget.value)} />
307+
<TimeInput
308+
error={isStartTimeMoreThenEndTime && 'Start time must be earlier than end time.'}
309+
value={startingTime}
310+
onChange={(e) => onTimeSelect('startTime', e.currentTarget.value)}
311+
/>
308312
</Stack>
309313
<Stack className={classes.datePickerContainer}>
310314
<DatePicker

0 commit comments

Comments
 (0)