Skip to content

Fix/forms 8364 clear initialvalue #8367

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
Matthias-dmt wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix/forms 8364 clear initialvalue #8367

Matthias-dmt wants to merge 2 commits into from
+50 −3

Conversation

@Matthias-dmt
Copy link

@Matthias-dmt Matthias-dmt commented Jan 9, 2026 •
edited
Loading

Fixes #8364 & #7945

Summary

  • Prevent BaseEditableHolder from reapplying form initialValue after a field is cleared.
  • Add regression test for InputNumber used inside FormField with initialValue.

Details

  • In BaseEditableHolder, the form default value is now applied only when the component is uninitialized (d_value === undefined).
  • This keeps null as a valid "cleared" state for form fields.

Test

  • pnpm exec vitest run src/inputnumber/InputNumber.spec.js

@Mettbrot
Copy link

Mettbrot commented Jan 12, 2026

also fixes duplicate #7945

@Mettbrot
Copy link

Mettbrot commented Jan 19, 2026

There is also an older bug #7633 with a fix here: #8036

How does this compare?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

InputNumber cannot be cleared inside Form when initialValues are set

2 participants

@Matthias-dmt @Mettbrot