Skip to content

docs(material/form-field): fix incorrect chip component reference#33161

Merged
crisbeto merged 1 commit into
angular:mainfrom
Will-thom:fix/form-field-chip-doc
Jun 19, 2026
Merged

docs(material/form-field): fix incorrect chip component reference#33161
crisbeto merged 1 commit into
angular:mainfrom
Will-thom:fix/form-field-chip-doc

Conversation

@Will-thom

Copy link
Copy Markdown
Contributor

Fix incorrect chips component in mat-form-field documentation

The documentation currently lists mat-chip-set as compatible with mat-form-field, but it does not implement MatFormFieldControl and results in the error:

"mat-form-field must contain a MatFormFieldControl"

This PR updates the documentation to use mat-chip-grid, which correctly implements the required interface.

Additionally, a note was added to clarify that mat-form-field requires a component implementing MatFormFieldControl.

Closes #33035

@google-cla

google-cla Bot commented Apr 28, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@pullapprove pullapprove Bot requested review from adolgachev and crisbeto April 28, 2026 17:03
@angular-robot angular-robot Bot added area: docs Related to the documentation area: material/form-field labels Apr 28, 2026
@Will-thom Will-thom force-pushed the fix/form-field-chip-doc branch from e95ae12 to 03518a9 Compare April 28, 2026 17:25
Replace mat-chip-set with mat-chip-grid in the form-field documentation,
since only mat-chip-grid implements MatFormFieldControl.

Closes angular#33035
@Will-thom Will-thom force-pushed the fix/form-field-chip-doc branch from 03518a9 to ea79898 Compare April 28, 2026 18:49
@Will-thom

Will-thom commented Apr 28, 2026

Copy link
Copy Markdown
Contributor Author

Hi @adolgachev or @crisbeto I've signed the CLA. Please let me know if anything else is needed 🙂

@tjshiu tjshiu added the docs: preview When applied, a preview of the documentation site is deployed to Firebase label May 15, 2026
@crisbeto crisbeto added target: patch This PR is targeted for the next patch release action: merge The PR is ready for merge by the caretaker labels Jun 19, 2026
@crisbeto crisbeto merged commit d829d1c into angular:main Jun 19, 2026
11 of 13 checks passed
@crisbeto

Copy link
Copy Markdown
Member

This PR was merged into the repository. The changes were merged into the following branches:

crisbeto pushed a commit that referenced this pull request Jun 19, 2026
…3161)

Replace mat-chip-set with mat-chip-grid in the form-field documentation,
since only mat-chip-grid implements MatFormFieldControl.

Closes #33035

Co-authored-by: Will-thom <shortgamestests@gmail.com>
(cherry picked from commit d829d1c)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: docs Related to the documentation area: material/form-field docs: preview When applied, a preview of the documentation site is deployed to Firebase target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs-bug(COMPONENT): Mat-Form-Field

4 participants