Feat/schedule unschedule due date - #112
Merged
Merged
Conversation
Replaces the read-only due date chip with a date input so the user can change the due date directly in the panel. The × button next to the input still clears the date, removes the time slot, and unschedules. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Switches the Notes textarea from a live-update controlled input to
local state saved on blur. key={block.id} on EditPanel resets the
local state when a different block is opened.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
blur-to-save was unreliable when switching blocks by clicking calendar divs (non-focusable, so blur never fired). Refs track the latest notes and onUpdateTimeBlock; a cleanup effect flushes on unmount regardless of how the panel closes. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Replaces auto-save-on-blur with an explicit Save button that appears when text is entered. After saving, notes render read-only with an Edit button to re-enter edit mode. Defaults to edit mode when no notes exist. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
onUpdateTimeBlock closes over editBlock in the parent which could be stale at save time. EditPanel now reads updateTimeBlock from the store directly and uses block.id/sourceType from its own props — both are stable for the lifetime of the panel (key=block.id). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
updateTask reconstructs the linked TimeBlock from scratch (lines
1118-1131 in store.ts) without preserving notes, silently wiping them
on any category/title/time change. Moving notes to Task.notes fixes
this: Task updates merge with {...current, ...updates}, so fields
absent from updates are always preserved.
Non-linked time blocks keep notes on TimeBlock as before.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Why
How
How to Test
Checklist
npm run buildpasses with zero type errorsnpm run lintpassesconsole.logleft in production codeScreenshots (if UI change)