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

Commit d8ed637

Browse files
committed
fix: disable decrease button when days worked is 0
ref issue #37
1 parent b388ec7 commit d8ed637

File tree

1 file changed

+3
-2
lines changed
  • src/routes/WorkPeriods/components/PeriodWorkingDays

1 file changed

+3
-2
lines changed

src/routes/WorkPeriods/components/PeriodWorkingDays/index.jsx

+3-2
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,8 @@ const PeriodWorkingDays = ({
3636
onWorkingDaysUpdateHintTimeout,
3737
updateHintTimeout = 2000,
3838
}) => {
39-
const isBtnMinusDisabled = daysWorked > 0 && daysWorked <= daysPaid;
39+
const isBtnMinusDisabled =
40+
daysWorked === 0 || (daysWorked > 0 && daysWorked <= daysPaid);
4041
const isBtnPlusDisabled = daysWorked < 5 && daysWorked >= daysWorkedMax;
4142
const decreaseDaysWorkedMessage = useMemo(
4243
() => `Cannot decrease "Working Days" below the number of days already
@@ -80,7 +81,7 @@ const PeriodWorkingDays = ({
8081
})}
8182
tooltipClassName={styles.tooltip}
8283
content={decreaseDaysWorkedMessage}
83-
isDisabled={!isBtnMinusDisabled || isDisabled}
84+
isDisabled={!isBtnMinusDisabled || isDisabled || daysWorked === 0}
8485
strategy="fixed"
8586
>
8687
<button

0 commit comments

Comments
 (0)