jonas/donetick
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

Merge pull request #79 from hoborm/bugfix/rework-create-chore-notification-input-v2

Browse Source 
Fix issue where cannot delete input field before input
This commit is contained in:
Mohamad Tarbin
2026-04-02 23:54:25 -04:00
committed by GitHub
parent 6ab6d0d205 0a606f6d23
commit bfa5c66006
1 changed files with 9 additions and 29 deletions
Download Patch File Download Diff File Expand all files Collapse all files

38
src/components/NotificationTemplate.jsx
View File

@@ -589,44 +589,24 @@ const NotificationTemplate = ({
const val = e.target.value
if (val.includes('-')) return
// Prevent setting to '0' if an 'On Due' already exists elsewhere
if (val === '0' || val === '') {
if (hasOnDueElsewhere) return
}
setDraftValues(prev => ({ ...prev, [idx]: val }))
}}
onKeyDown={e => {
if (['-', 'e', '+', '.'].includes(e.key)) {
e.preventDefault()
return
}
// Physically block typing '0' if 'On Due' exists
if (
e.key === '0' ||
e.key === 'Backspace' ||
e.key === 'Delete'
) {
const currentVal =
draftValues[idx] !== undefined
? draftValues[idx]
: uiRep.displayValue.toString()
// If typing 0 into an empty input, or deleting the last character
const willBeZero =
(e.key === '0' && currentVal === '') ||
((e.key === 'Backspace' || e.key === 'Delete') &&
currentVal.length <= 1)
if (willBeZero && hasOnDueElsewhere) {
e.preventDefault()
}
}
}}
onBlur={e => {
let val = e.target.value
if (val === '' || Number(val) < 0) val = 0
const numericVal = Number(val)
val =
numericVal <= 0
? hasOnDueElsewhere
? 1
: 0
: numericVal
handleChange(idx, 'displayValue', val)
setDraftValues(prev => {
const next = { ...prev }