-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
fix(material/list): Mat Selection List example is limited to html temp #29348
fix(material/list): Mat Selection List example is limited to html temp #29348
Conversation
…plating Updated the single selection list example from a template-driven form to a reactive form. Fixes angular#25894
<p> | ||
Option selected: {{shoes.selectedOptions.hasValue() ? shoes.selectedOptions.selected[0].value : 'None'}} | ||
</p> | ||
<form [formGroup]="form"> |
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.
Rather than replacing the example, I think we should add another one with reactive forms.
name: string; | ||
} | ||
/** | ||
* @title List with single selection using a reactive forms approach |
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.
using a reactive forms approach -> using Reactive forms
@@ -0,0 +1,33 @@ | |||
import {Component} from '@angular/core'; | |||
import {FormControl, FormGroup, FormsModule, ReactiveFormsModule} from '@angular/forms'; | |||
import {MatListModule} from '@angular/material/list'; |
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.
nit: add a new line after this import.
@MichelleRetroRabbit the change looks good, but I can't merge it in because the branch contains a merge commit. Can you drop remove it? |
…plating Added a reactive forms example to the single selection list Fixes angular#25894
f6de090
to
3bdaaa4
Compare
…mp (#29348) * fix(material/list): Mat Selection List example is limited to html templating Updated the single selection list example from a template-driven form to a reactive form. Fixes #25894 * fix(material/list): Mat Selection List example is limited to html templating Added a reactive forms example to the single selection list Fixes #25894 (cherry picked from commit 2f49eaa)
…mp (#29348) * fix(material/list): Mat Selection List example is limited to html templating Updated the single selection list example from a template-driven form to a reactive form. Fixes #25894 * fix(material/list): Mat Selection List example is limited to html templating Added a reactive forms example to the single selection list Fixes #25894 (cherry picked from commit 2f49eaa)
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. |
Updated the single selection list example from a template-driven form to a reactive form.
Fixes #25894