From 6ac0f3799a55da56f901f37dd74c03a219edf4be Mon Sep 17 00:00:00 2001 From: Bernt Christian Egeland Date: Mon, 23 Jan 2023 20:20:44 +0100 Subject: [PATCH] do not add hours if before is zero --- web/src/components/TimePicker.jsx | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/web/src/components/TimePicker.jsx b/web/src/components/TimePicker.jsx index eadceb5cb..70a3727fb 100644 --- a/web/src/components/TimePicker.jsx +++ b/web/src/components/TimePicker.jsx @@ -23,9 +23,8 @@ const TimePicker = ({ dateRange, onChange }) => { }, [dateRange]); useEffect(() => { - if (reset) return; + if (reset || !before) return; - if (!after || !before) return; if (before.getHours() === 0 && after.getHours() === 0) return setTimeRange(new Set()); /** @@ -34,7 +33,7 @@ const TimePicker = ({ dateRange, onChange }) => { */ const days = Math.max(before.getDate() - after.getDate()); const hourOffset = days * 24; - const beforeOffset = hourOffset + before.getHours(); + const beforeOffset = before.getHours() ? hourOffset + before.getHours() : 0; /** * Fills the timeRange by iterating over the hours between 'after' and 'before' during component mount, to keep the selected hours persistent. @@ -156,7 +155,6 @@ const TimePicker = ({ dateRange, onChange }) => { }, [after, before, timeRange, dateRange.after, numberOfDaysSelected.size, onChange] ); - const isSelected = useCallback( (idx) => { return !!timeRange.has(idx);