-
Notifications
You must be signed in to change notification settings - Fork 6.8k
docs(material/select): add harness example for select #21464
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
Conversation
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 once the comments have been addressed.
// Angular won't call this for us so we need to do it ourselves to avoid leaks. | ||
overlayContainer.ngOnDestroy(); | ||
overlayContainer = null!; | ||
}); |
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.
We can drop this afterEach
and the dependency on the OverlayContainer
since it was added primarily for our own unit tests.
@@ -0,0 +1,21 @@ | |||
import {Component} from '@angular/core'; | |||
|
|||
interface Food { |
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.
I think that you can remove this interface, TS will infer the type itself.
6b414dc
to
d39061b
Compare
d39061b
to
95ddae9
Compare
(cherry picked from commit dc5b773)
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. |
No description provided.