-
Notifications
You must be signed in to change notification settings - Fork 24.8k
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.
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
docs(forms): make the reactive forms guide specify more clearly discuss mixing HTML5 native validators with Angular forms. #42377
Conversation
|
137f597
to
336327f
Compare
77cc58e
to
56701be
Compare
f60eba8
to
b028ca0
Compare
You can preview 336327f at https://pr42377-336327f.ngbuilds.io/. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
…t happens when mixing HTML5 native validators with Angular forms. Previously, the docs didn't say anything about the fact that the `novalidate` attribute is added to the enclosing form, or how to override that behavior. I have added a couple lines in the appropriate spot clarifying this issue.
You can preview 6c87908 at https://pr42377-6c87908.ngbuilds.io/. |
…t happens when mixing HTML5 native validators with Angular forms. (#42377) Previously, the docs didn't say anything about the fact that the `novalidate` attribute is added to the enclosing form, or how to override that behavior. I have added a couple lines in the appropriate spot clarifying this issue. PR Close #42377
…t happens when mixing HTML5 native validators with Angular forms. (angular#42377) Previously, the docs didn't say anything about the fact that the `novalidate` attribute is added to the enclosing form, or how to override that behavior. I have added a couple lines in the appropriate spot clarifying this issue. PR Close angular#42377
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
I have added a couple lines in the appropriate spot clarifying this issue.
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Reactive forms allow the use of HTML5 native validators, but there is a gotcha:
novalidate
is added to the enclosing form, disabling the native HTML5 behavior. Previously, the docs didn't say anything about that, or how to override that behavior.Issue Number: 39549
What is the new behavior?
The docs now specify that
novalidate
is added to the enclosing form, and native validation can be re-enabled with thengNativeValidate
directive.Does this PR introduce a breaking change?
Other information
Closes #39549