fix: preserve date when manually entering the year

[FranklinD10]

What this fixes

Closes #5072.

When manually typing a date, tabbing into or editing the year field made the whole entry disappear. While the year section is incomplete, MUI emits an Invalid Date, and the picker was forwarding that straight to the parent onChange, which wiped the stored value (this also caused the day/month to reset when editing the year).

The change

In handleValueChange (src/components/date_picker/index.tsx), only call the parent onChange when the value is a complete valid date or the field has been fully cleared (null). Intermediate Invalid Date states are kept local to the field, so partial typing no longer clears anything. The clear button still passes null through as before.

Testing

• Reproduced with a person's Time away end date and the CO visit date: typing DD, MM, then YYYY no longer clears the field.
• Fully clearing the field still works.
• npm run build passes and eslint is clean on the changed file.

Scope is intentionally kept to the disappearing-value bug. The other UX points in the issue (focus retention, widget max-height, end-date-before-start validation, widget opening on the start month) are separate and can be follow-ups.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
staging-organized-app	Ready	Preview	Jul 26, 2026 7:21pm
test-organized-app	Ready	Preview	Jul 26, 2026 7:21pm

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 10845eb4-991f-4c70-9730-c4c8c93813f0

📥 Commits

Reviewing files that changed from the base of the PR and between 3cff27b and 18bdbab.

📒 Files selected for processing (1)

• src/components/date_picker/index.tsx

---

Walkthrough

The date picker now prevents invalid intermediate manual entries from reaching the parent, while still propagating fully valid dates and cleared values.

Changes

Date input validation

Layer / File(s)	Summary
Guard date change propagation src/components/date_picker/index.tsx	handleValueChange invokes onChange only for valid Date values or null, excluding invalid intermediate input.

Estimated code review effort: 1 (Trivial) | ~5 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes the main fix: preserving dates during manual year entry.
Description check	✅ Passed	The description matches the change and explains the manual date-entry bug fix.
Linked Issues check	✅ Passed	The code prevents intermediate Invalid Date values from reaching the parent, matching #5072's manual YYYY entry fix.
Out of Scope Changes check	✅ Passed	The change is narrowly scoped to the date-picker bug and introduces no unrelated changes.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[FranklinD10]

@rhahao @ux-git

[FranklinD10]

@fatmarker can you test this now works in https://staging-organized-app-git-fork-franklind10-fix-a2c43c-sws2apps.vercel.app/

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud