fix: don't lose tags typed before tapping Done #25

Merged

rssnyder merged 1 commit from fix/tag-save into main

2026-08-12 10:19:20 -05:00

rssnyder commented

2026-08-12 10:19:00 -05:00

Owner

Two ways an added tag appeared not to save:

TagInput only committed a tag on Add/return-key, so text still sitting in
the field was discarded when the editor closed. It now commits on blur,
which covers Done on the event screen and Start Timer on the new-timer
sheet. addTag() no-ops on an empty or duplicate draft, so submit-then-blur
can't double-add.

EventDetail also re-read the event from the API on every focus and rendered
that over the local row, so an edit still sitting in the sync queue visibly
reverted. Pending local metadata is now overlaid on the server response
while the row is dirty, and the screen flushes the queue on focus so the
server catches up.

Co-Authored-By: Claude Opus 5 noreply@anthropic.com

Two ways an added tag appeared not to save: TagInput only committed a tag on Add/return-key, so text still sitting in the field was discarded when the editor closed. It now commits on blur, which covers Done on the event screen and Start Timer on the new-timer sheet. addTag() no-ops on an empty or duplicate draft, so submit-then-blur can't double-add. EventDetail also re-read the event from the API on every focus and rendered that over the local row, so an edit still sitting in the sync queue visibly reverted. Pending local metadata is now overlaid on the server response while the row is dirty, and the screen flushes the queue on focus so the server catches up. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

rssnyder added 1 commit

2026-08-12 10:19:01 -05:00

fix: don't lose tags typed before tapping Done

Backend CI / test (pull_request) Successful in 17s

Details

Backend CI / build-push-api (pull_request) Successful in 30s

Details

Backend CI / build-push-web (pull_request) Successful in 21s

Details

4ffd5e6a12

Two ways an added tag appeared not to save:

TagInput only committed a tag on Add/return-key, so text still sitting in
the field was discarded when the editor closed. It now commits on blur,
which covers Done on the event screen and Start Timer on the new-timer
sheet. addTag() no-ops on an empty or duplicate draft, so submit-then-blur
can't double-add.

EventDetail also re-read the event from the API on every focus and rendered
that over the local row, so an edit still sitting in the sync queue visibly
reverted. Pending local metadata is now overlaid on the server response
while the row is dirty, and the screen flushes the queue on focus so the
server catches up.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

rssnyder merged commit 3f3180da5e into main

2026-08-12 10:19:20 -05:00

rssnyder deleted branch fix/tag-save

2026-08-12 10:19:21 -05:00

rssnyder referenced this pull request from a commit

2026-08-12 10:19:22 -05:00

Merge pull request 'fix: don't lose tags typed before tapping Done' (#25) from fix/tag-save into main