8000 fix(forms): only touch visible, interactive fields on submit by leonsenft · Pull Request #66785 · angular/angular · GitHub
[go: up one dir, main page]

Skip to content

fix(forms): only touch visible, interactive fields on submit#66785

Merged
atscott merged 1 commit intoangular:mainfrom
leonsenft:sf-submit-touch-visible-fields
Jan 28, 2026
Merged

fix(forms): only touch visible, interactive fields on submit#66785
atscott merged 1 commit intoangular:mainfrom
leonsenft:sf-submit-touch-visible-fields

Conversation

@leonsenft
Copy link
Contributor

Don't touch hidden, disabled, or readonly fields on submit, since they don't contribute to form validity. This also prevents errors from appearing immediately if they're later made interactive.

Fix #66344

Don't touch hidden, disabled, or readonly fields on submit, since they
don't contribute to form validity. This also prevents errors from
appearing immediately if they're later made interactive.

Fix angular#66344
@ngbot ngbot bot added this to the Backlog milestone Jan 28, 2026
@leonsenft leonsenft added the target: patch This PR is targeted for the next patch release label Jan 28, 2026
@leonsenft leonsenft marked this pull request as ready for review January 28, 2026 04:34
@leonsenft leonsenft requested review from kirjs and mmalerba January 28, 2026 04:34
@leonsenft leonsenft added action: merge The PR is ready for merge by the caretaker and removed area: forms labels Jan 28, 2026
@ngbot ngbot bot modified the milestone: Backlog Jan 28, 2026
@ngbot ngbot bot added this to the Backlog milestone Jan 28, 2026
@leonsenft leonsenft removed the request for review from kirjs January 28, 2026 18:37
@atscott atscott merged commit e682e53 into angular:main Jan 28, 2026
28 checks passed
@atscott
Copy link
Contributor
atscott commented Jan 28, 2026

This PR was merged into the repository. The changes were merged into the following branches:

@leonsenft leonsenft deleted the sf-submit-touch-visible-fields branch January 29, 2026 21:39
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Mar 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: forms forms: signals target: patch This PR is targeted for the next patch release

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Signal Forms: Mark all as touched when not hidden

3 participants

0