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(select): emitting change event twice for reset values #13598
fix(select): emitting change event twice for reset values #13598
Conversation
5d1bb3c
to
f3120bf
Compare
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
f3120bf
to
55dbbf4
Compare
55dbbf4
to
1dad2a8
Compare
Anything make this pr can not be merged? |
Can this be merged? |
1dad2a8
to
51a8079
Compare
Can this be merged PLZ ! |
When this PR will be merged? |
Bumping thing to a P2 since it keeps coming up. |
Why don't merge and fix the issue ? |
51a8079
to
863b61c
Compare
We're seeing internal failures and I believe I was able to reproduce the issue. See this PR for the failing test reproduction. In short, the select's |
5dfe4d2
to
ce432f4
Compare
Updated to cover the extra test cases @andrewseguin. |
Fixes `mat-select` emitting its change event twice when a reset value is selected, as well as when it's selected twice in a row. This PR covers angular#10859 which would've introduced another issue. Fixes angular#10675. Fixes angular#13579.
ce432f4
to
2eb04d0
Compare
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. |
Fixes
mat-select
emitting its change event twice when a reset value is selected, as well as when it's selected twice in a row. This PR covers #10859 which would've introduced another issue.Fixes #10675.
Fixes #13579.