fix(material/chips): update form control immediately when chips is removed by mistrykaran91 · Pull Request #31420 · angular/components · GitHub | Latest TMZ Celebrity News & Gossip | Watch TMZ Live
Skip to content

fix(material/chips): update form control immediately when chips is removed #31420

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mistrykaran91
Copy link
Contributor

Currently, when we removed chips, the value is updated in form control only when we blur. This fix will update the value of form control immediately when removed

Fixes #30566

@mistrykaran91 mistrykaran91 requested a review from a team as a code owner June 24, 2025 11:51
@mistrykaran91 mistrykaran91 requested review from crisbeto and ok7sai and removed request for a team June 24, 2025 11:51
@mistrykaran91 mistrykaran91 force-pushed the feat/update-form-value-on-chip-removal branch from 8181759 to 3927001 Compare June 24, 2025 12:00
@crisbeto
Copy link
Member

This seems identical to #30818.

@mistrykaran91
Copy link
Contributor Author

mistrykaran91 commented Jun 24, 2025

This seems identical to #30818.

This one only includes remove chips, #30818 was stuck due to some internal test failure while add chips updation.
So, I had created this one with only remove and added unit test for the same.

…moved

Currently, when we removed chips, the value is updated in form control only when we blur. This fix will update the value of form control immediately when removed

Fixes angular#30566
@mistrykaran91 mistrykaran91 force-pushed the feat/update-form-value-on-chip-removal branch from 3927001 to 2fd3421 Compare June 27, 2025 10:03
@mistrykaran91
Copy link
Contributor Author

@crisbeto: Do you want me to close this MR, if that is to be merged with #30818. I'm not sure when that will be merged or where it's failing so that was the reason I created separate MR to include only remove feature which is this bug

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat(MatChipGrid): Update form component value on chip removal
2 participants

TMZ Celebrity News – Breaking Stories, Videos & Gossip

Looking for the latest TMZ celebrity news? You've come to the right place. From shocking Hollywood scandals to exclusive videos, TMZ delivers it all in real time.

Whether it’s a red carpet slip-up, a viral paparazzi moment, or a legal drama involving your favorite stars, TMZ news is always first to break the story. Stay in the loop with daily updates, insider tips, and jaw-dropping photos.

🎥 Watch TMZ Live

TMZ Live brings you daily celebrity news and interviews straight from the TMZ newsroom. Don’t miss a beat—watch now and see what’s trending in Hollywood.