From af450f651ade758331e4e44555084fd15a012d31 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Luis=20F=C3=A9lix?= Date: Tue, 19 Nov 2019 16:46:02 +0000 Subject: [PATCH] fix: use returns instead of elses --- .../date-range-picker/DateRangePicker.js | 21 +++++++++++-------- .../src/components/date-range-picker/Tile.js | 1 - 2 files changed, 12 insertions(+), 10 deletions(-) diff --git a/new-lamassu-admin/src/components/date-range-picker/DateRangePicker.js b/new-lamassu-admin/src/components/date-range-picker/DateRangePicker.js index 8b2b0dfb..11f560f0 100644 --- a/new-lamassu-admin/src/components/date-range-picker/DateRangePicker.js +++ b/new-lamassu-admin/src/components/date-range-picker/DateRangePicker.js @@ -25,17 +25,20 @@ const DateRangePicker = ({ minDate, maxDate, className, onRangeChange, ...props const handleSelect = (day, minDate, maxDate) => { if ((maxDate && day.isAfter(maxDate, 'day')) || (minDate && day.isBefore(minDate, 'day'))) return - if (from && !to) { - if (day.isBefore(from, 'day')) { - setTo(from) - setFrom(day) - } else { - setTo(day) - } - } else { + + if (from && !to && day.isBefore(from, 'day')) { + setTo(from) setFrom(day) - setTo(null) + return } + + if (from && !to && day.isSameOrAfter(from, 'day')) { + setTo(day) + return + } + + setFrom(day) + setTo(null) } return ( diff --git a/new-lamassu-admin/src/components/date-range-picker/Tile.js b/new-lamassu-admin/src/components/date-range-picker/Tile.js index 395443db..d6e2e0c8 100644 --- a/new-lamassu-admin/src/components/date-range-picker/Tile.js +++ b/new-lamassu-admin/src/components/date-range-picker/Tile.js @@ -9,7 +9,6 @@ const { label1 } = typographyStyles const styles = { wrapper: { - height: 26, display: 'flex', alignItems: 'center',