added onUserInteractionIfError for form #175515

akashefrath · 2025-09-17T13:50:24Z

This PR adds a new AutovalidateMode value:
onUserInteractionIfError.

This mode allows Form and FormField widgets to auto-validate only when a field already has an error and the user interacts with it again.

Why

Current modes (disabled, always, onUserInteraction, onUnfocus) do not cover the case where developers want validation only when correcting an error.
This improves UX by reducing unnecessary validation calls while still ensuring errors disappear as soon as the user fixes them.

How

Added onUserInteractionIfError to AutovalidateMode.
Updated Dartdoc with detailed description and example snippet.
Added new unit tests to validate behavior.
Verified no regressions in other modes (always, onUnfocus, onUserInteraction).

Example

TextFormField(
  autovalidateMode: AutovalidateMode.onUserInteractionIfError,
  validator: (value) => value!.isEmpty ? 'Required field' : null,
)

Related Issues

Fixes #<INSERT_RELATED_ISSUE_NUMBER_IF_ANY>

Tests

Added widget tests to confirm that:
- No validation occurs until a field has an error.
- Once an error exists, validation runs automatically on user interaction.
- Resetting the form clears errors correctly.
Regression tests ensure that always, onUserInteraction, and onUnfocus modes still behave correctly.

Pre-launch / Reviewer Checklist

Breaking Change

Yes
No

Screenshots / Demo (if applicable)

(N/A — logic-only change)

Reviewer Notes

The new mode reduces unnecessary validation until the user corrects an error.
All tests confirm correct behavior and no regressions in existing modes.

fluttergithubbot · 2025-09-17T13:50:28Z

An existing Git SHA, 7a01f9e517cba39fc283da4664da4abed47aad4f, was detected, and no actions were taken.

To re-trigger presubmits after closing or re-opeing a PR, or pushing a HEAD commit (i.e. with --force) that already was pushed before, push a blank commit (git commit --allow-empty -m "Trigger Build") or rebase to continue.

gemini-code-assist

Code Review

This pull request introduces a new AutovalidateMode called onUserInteractionIfError. This mode triggers form field validation only when a user interacts with a field that already has an error. The implementation adds the new enum value, updates the logic in FormState and FormFieldState to handle this new mode, and includes comprehensive unit tests to verify the new behavior and prevent regressions. The changes are logical and well-implemented. My feedback includes a couple of minor style suggestions to improve code conciseness by leveraging Dart's type inference, in line with the Effective Dart style guide.

packages/flutter/lib/src/widgets/form.dart

packages/flutter/test/widgets/form_test.dart

justinmc

I think that the real, long-term solution here is to give users more control over when errors are displayed. I have been meaning to work on this myself for years... But given that I haven't gotten around to it, I'm up for adding things like this for realistic use cases that aren't possible today. I think this one qualifies. Thanks for the PR.

packages/flutter/lib/src/widgets/form.dart

packages/flutter/test/widgets/form_test.dart

packages/flutter/lib/src/widgets/form.dart

packages/flutter/test/widgets/form_test.dart

…eractionIfError

akashefrath · 2025-10-22T08:46:36Z

I think that the real, long-term solution here is to give users more control over when errors are displayed. I have been meaning to work on this myself for years... But given that I haven't gotten around to it, I'm up for adding things like this for realistic use cases that aren't possible today. I think this one qualifies. Thanks for the PR.

Thanks so much for the thoughtful feedback! I completely agree — giving users more control over error display timing would definitely make validation more flexible and intuitive. I’m glad this use case aligns with that direction, and really appreciate your openness to the change.

And will fix the suggested changes,

justinmc

Thanks for the fixes! Two main things here: a use of a deprecated member, and some minor suggestions for improving the tests.

packages/flutter/lib/src/widgets/form.dart

packages/flutter/test/widgets/form_test.dart

Renzo-Olivares · 2025-12-05T00:27:19Z

packages/flutter/lib/src/widgets/form.dart

  int _generation = 0;
  bool _hasInteractedByUser = false;
  final Set<FormFieldState<dynamic>> _fields = <FormFieldState<dynamic>>{};
-


nit: unnecessary deletion? Or is this from dart format?

Noted, thanks! I only added back an empty line.

Renzo-Olivares

LGTM w/ small nits. Thank you for the contribution!

Piinks · 2025-12-09T23:11:57Z

It looks like this is ready to land, I am just going to rebase it first. Thanks for the contribution!

Piinks · 2025-12-09T23:12:14Z

Oh! It looks like I cannot rebase, can you take a look @akashefrath?

akashefrath · 2025-12-10T00:03:39Z

Oh! It looks like I cannot rebase, can you take a look @akashefrath?

Done. CI running .

Piinks · 2025-12-10T00:08:44Z

Thank you!

auto-submit · 2025-12-10T01:26:14Z

autosubmit label was removed for flutter/flutter/175515, because - The status or check suite Google testing has failed. Please fix the issues identified (or deflake) before re-applying this label.

akashefrath · 2025-12-10T03:11:48Z

@Piinks The autosubmit label was removed because the Google testing check failed earlier. I’ve now rebased the PR and re-tested, and currently only Tree Status is red; all other required checks are passing.

The remaining tree status failure is unrelated to this change. Could you please re-apply the autosubmit label to #175515?

Thank you

…5515)

…10601) Manual roll Flutter from 6a1f5b7f85a4 to e25d71b086d6 (30 revisions) Manual roll requested by [email protected] flutter/flutter@6a1f5b7...e25d71b 2025-12-10 [email protected] Ensure that the engine converts std::filesystem::path objects to UTF-8 strings on Windows (flutter/flutter#179528) 2025-12-10 [email protected] Fix the issue with pinned headers in nested SliverMainAxisGroup. (flutter/flutter#179132) 2025-12-10 [email protected] added onUserInteractionIfError for form (flutter/flutter#175515) 2025-12-10 [email protected] Fixed RenderFlex overflow in RouteObserver Example (flutter/flutter#170980) 2025-12-10 [email protected] Roll Dart SDK from 17749965ec57 to 077062c5e515 (3 revisions) (flutter/flutter#179691) 2025-12-10 [email protected] Manually roll characters (flutter/flutter#179447) 2025-12-10 [email protected] Roll Packages from 338ecd3 to 74a5a53 (4 revisions) (flutter/flutter#179693) 2025-12-10 [email protected] Marks Mac_ios draw_arcs_all_stroke_styles_perf_ios__timeline_summary to be unflaky (flutter/flutter#179669) 2025-12-10 [email protected] Check for a null cached image in SingleFrameCodec::getNextFrame (flutter/flutter#179483) 2025-12-10 [email protected] Roll Fuchsia Linux SDK from _pSztGZvEA3-Ry-GW... to u5vxWTRT0HlxOP5_r... (flutter/flutter#179652) 2025-12-10 [email protected] Implement flutter/accessibility channel (flutter/flutter#179484) 2025-12-10 [email protected] Roll Skia from 82fff05cc621 to e61cc6d073fd (4 revisions) (flutter/flutter#179646) 2025-12-10 [email protected] Make sure that a CupertinoDialogAction doesn't crash in 0x0 environment (flutter/flutter#178956) 2025-12-10 [email protected] Make SettingsChannel configuration queue not static (flutter/flutter#179636) 2025-12-10 [email protected] Make sure that a CupertinoListSection doesn't crash in 0x0 environment (flutter/flutter#179068) 2025-12-10 [email protected] Make sure that a CupertinoFormSection doesn't crash in 0x0 environment (flutter/flutter#179001) 2025-12-10 [email protected] Make sure that a CupertinoMagnifier doesn't crash in 0x0 environment (flutter/flutter#179206) 2025-12-10 [email protected] Make sure that a Tooltip doesn't crash in 0x0 environment (flutter/flutter#178461) 2025-12-10 [email protected] Make sure that a CupertinoSegmentedControl doesn't crash in 0x0 envir… (flutter/flutter#179544) 2025-12-10 [email protected] Make sure that a CupertinoSlider doesn't crash in 0x0 environment (flutter/flutter#179566) 2025-12-10 [email protected] Make sure that a CupertinoPageScaffold doesn't crash in 0x0 environment (flutter/flutter#179245) 2025-12-09 [email protected] Roll Skia from f9e32c28c5c5 to 82fff05cc621 (2 revisions) (flutter/flutter#179641) 2025-12-09 [email protected] Roll Dart SDK from 019cb923bf62 to 17749965ec57 (5 revisions) (flutter/flutter#179640) 2025-12-09 [email protected] Enhance documentation for `LocalHistoryEntry` class (flutter/flutter#179223) 2025-12-09 [email protected] WebParagrah: ellipsis (flutter/flutter#178748) 2025-12-09 [email protected] Update the doc on Google Testing to reflect the current state (flutter/flutter#177187) 2025-12-09 [email protected] [wimp] Initial Impeller on Web implementation. (flutter/flutter#175442) 2025-12-09 [email protected] Relax assertion for adding semantics locale (flutter/flutter#178140) 2025-12-09 [email protected] Fix Scrollbar drag behavior (flutter/flutter#179199) 2025-12-09 [email protected] Roll Skia from 502ee6f2a0d7 to f9e32c28c5c5 (6 revisions) (flutter/flutter#179632) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md ...

This PR adds a new `AutovalidateMode` value: `onUserInteractionIfError`. This mode allows `Form` and `FormField` widgets to auto-validate **only when a field already has an error and the user interacts with it again**. ### Why - Current modes (`disabled`, `always`, `onUserInteraction`, `onUnfocus`) do not cover the case where developers want validation **only when correcting an error**. - This improves UX by reducing unnecessary validation calls while still ensuring errors disappear as soon as the user fixes them. ### How - Added `onUserInteractionIfError` to `AutovalidateMode`. - Updated Dartdoc with detailed description and example snippet. - Added new unit tests to validate behavior. - Verified no regressions in other modes (`always`, `onUnfocus`, `onUserInteraction`). ### Example ```dart TextFormField( autovalidateMode: AutovalidateMode.onUserInteractionIfError, validator: (value) => value!.isEmpty ? 'Required field' : null, ) ```` --- ## Related Issues Fixes #\<INSERT\_RELATED\_ISSUE\_NUMBER\_IF\_ANY> --- ## Tests * Added widget tests to confirm that: * No validation occurs until a field has an error. * Once an error exists, validation runs automatically on user interaction. * Resetting the form clears errors correctly. * Regression tests ensure that `always`, `onUserInteraction`, and `onUnfocus` modes still behave correctly. --- ## Pre-launch / Reviewer Checklist * [x] I have read the [[Contributor Guide](https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md)](https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md) * [x] I have read the \[Tree Hygiene] wiki page and followed its guidance * [x] I have read and followed the \[Flutter Style Guide] * [x] I signed the \[CLA] * [x] I added a new enum value with clear Dartdoc (`onUserInteractionIfError`) * [x] I added example code in the Dartdoc * [x] I added new widget/unit tests for the new mode * [x] All existing and new tests pass * [x] I ran `flutter analyze` and fixed all issues * [x] No breaking changes introduced (or marked accordingly) --- ## Breaking Change * [x] Yes * [x] No --- ## Screenshots / Demo (if applicable) *(N/A — logic-only change)* --- ## Reviewer Notes * The new mode reduces unnecessary validation until the user corrects an error. * All tests confirm correct behavior and no regressions in existing modes. --------- Co-authored-by: Loïc Sharma <[email protected]>

akashefrath added 3 commits September 17, 2025 16:29

added onUserInteractionIfError for form

a59b61a

fix test error

91e3b16

fix dart analyze

7a01f9e

github-actions bot added the framework flutter/packages/flutter repository. See also f: labels. label Sep 17, 2025

Trigger Build

8b0602d

gemini-code-assist bot reviewed Sep 17, 2025

View reviewed changes

packages/flutter/lib/src/widgets/form.dart Outdated Show resolved Hide resolved

packages/flutter/test/widgets/form_test.dart Show resolved Hide resolved

akashefrath and others added 5 commits September 17, 2025 19:33

Trigger Build

e479259

Merge branch 'master' into feature/onUserInteractionIfError

bf04ca8

Merge branch 'master' into feature/onUserInteractionIfError

59ed5ce

Merge branch 'master' into feature/onUserInteractionIfError

b5163af

Merge branch 'master' into feature/onUserInteractionIfError

cc9649b

justinmc self-requested a review October 14, 2025 23:09

justinmc reviewed Oct 21, 2025

View reviewed changes

akashefrath added 4 commits October 22, 2025 10:15

update code style and fix form test

9080fa6

Merge remote-tracking branch 'upstream/master' into feature/onUserInt…

9d3d212

…eractionIfError

Trigger Build

b481ba3

fix getter

eb67d12

akashefrath and others added 5 commits October 22, 2025 14:22

Merge branch 'master' into feature/onUserInteractionIfError

a1fb68d

fix code Consistently

8df556f

fix error on test

7b96e7c

fix code formatter

ebed2cc

Merge branch 'master' into feature/onUserInteractionIfError

478416a

akashefrath requested a review from justinmc October 22, 2025 10:20

justinmc reviewed Oct 28, 2025

View reviewed changes

akashefrath added 4 commits October 30, 2025 17:12

Merge branch 'master' into feature/onUserInteractionIfError

90a4157

Merge branch 'master' into feature/onUserInteractionIfError

4ebb69d

Fix: Deprecated on from

1532195

Merge branch 'master' into feature/onUserInteractionIfError

3e90288

akashefrath added 3 commits December 4, 2025 11:23

Merge branch 'master' into feature/onUserInteractionIfError

1aa5d62

Merge branch 'master' into feature/onUserInteractionIfError

0cd052b

Merge branch 'master' into feature/onUserInteractionIfError

dce2e3c

Renzo-Olivares reviewed Dec 5, 2025

View reviewed changes

Renzo-Olivares approved these changes Dec 5, 2025

View reviewed changes

akashefrath and others added 2 commits December 5, 2025 10:58

fix:unnecessary deletion

c25bda1

Merge branch 'master' into feature/onUserInteractionIfError

d8609e6

Merge branch 'master' into feature/onUserInteractionIfError

2684471

Piinks added the autosubmit Merge PR when tree becomes green via auto submit App label Dec 10, 2025

auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Dec 10, 2025

Merge branch 'master' into feature/onUserInteractionIfError

14c5158

Merge branch 'master' into feature/onUserInteractionIfError

856f4d6

Piinks added the autosubmit Merge PR when tree becomes green via auto submit App label Dec 10, 2025

Merge branch 'master' into feature/onUserInteractionIfError

6e0c9b7

auto-submit bot added this pull request to the merge queue Dec 10, 2025

Merged via the queue into flutter:master with commit aaafeaf Dec 10, 2025
69 checks passed

flutter-dashboard bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Dec 10, 2025

engine-flutter-autoroll mentioned this pull request Dec 10, 2025

Manual roll Flutter from 6a1f5b7f85a4 to e25d71b086d6 (30 revisions) flutter/packages#10601

Merged

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 10, 2025

aaafeaf18 added onUserInteractionIfError for form (flutter/flutter#17…

6c5003b

…5515)

Piinks mentioned this pull request Dec 12, 2025

Flutter Notable Commits - 2025 #161460

Open

deandreamatias mentioned this pull request Dec 13, 2025

[General]: Support to new autovalidate mode flutter-form-builder-ecosystem/flutter_form_builder#1504

Open

1 task

added onUserInteractionIfError for form #175515

added onUserInteractionIfError for form #175515

Conversation

akashefrath commented Sep 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Why

How

Example

Related Issues

Tests

Pre-launch / Reviewer Checklist

Breaking Change

Screenshots / Demo (if applicable)

Reviewer Notes

Uh oh!

fluttergithubbot commented Sep 17, 2025

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

Uh oh!

Uh oh!

justinmc left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

akashefrath commented Oct 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

justinmc left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Renzo-Olivares Dec 5, 2025

Choose a reason for hiding this comment

Uh oh!

akashefrath Dec 5, 2025

Choose a reason for hiding this comment

Uh oh!

Renzo-Olivares left a comment

Choose a reason for hiding this comment

Uh oh!

Piinks commented Dec 9, 2025

Uh oh!

Piinks commented Dec 9, 2025

Uh oh!

akashefrath commented Dec 10, 2025

Uh oh!

Piinks commented Dec 10, 2025

Uh oh!

auto-submit bot commented Dec 10, 2025

Uh oh!

akashefrath commented Dec 10, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

akashefrath commented Sep 17, 2025 •

edited

Loading

akashefrath commented Oct 22, 2025 •

edited

Loading