Skip to content

fix: only disable SubmitButton while form is in submitting state#110192

Merged
TkDodo merged 1 commit intomasterfrom
tkdodo/ref/form-submit-behavior
Mar 10, 2026
Merged

fix: only disable SubmitButton while form is in submitting state#110192
TkDodo merged 1 commit intomasterfrom
tkdodo/ref/form-submit-behavior

Conversation

@TkDodo
Copy link
Collaborator

@TkDodo TkDodo commented Mar 9, 2026

the idea to also disable the SubmitButton if it's equal to the defaultValue falls apart for creation forms that have good default value set that you'd just want to submit. We want to keep the button enabled at all times, only disable it to avoid duplicate form submissions.

the idea to also disable the SubmitButton if it's equal to the defaultValue falls apart for creation forms that have good default value set that you'd just want to submit. We want to keep the button enabled at all times, only disable it to avoid duplicate form submissions.
@TkDodo TkDodo marked this pull request as ready for review March 9, 2026 07:35
@TkDodo TkDodo requested a review from a team as a code owner March 9, 2026 07:35
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 9, 2026
@TkDodo TkDodo merged commit 9b772fe into master Mar 10, 2026
61 checks passed
@TkDodo TkDodo deleted the tkdodo/ref/form-submit-behavior branch March 10, 2026 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants