Skip to content

Conversation

@calebegg
Copy link

Fixes #28067

@pullapprove pullapprove bot requested review from crisbeto and tjshiu January 20, 2026 20:52
@tjshiu tjshiu added dev-app preview When applied, previews of the dev-app are deployed to Firebase target: patch This PR is targeted for the next patch release docs: preview When applied, a preview of the documentation site is deployed to Firebase labels Jan 20, 2026
Copy link
Contributor

@tjshiu tjshiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So I like the change overall - but I do have a bit of a nit. The input is always on a new line here. Can we adjust it so that it isn't?

An alternative solution, but it would take more work: Add the role row and gridcell here and wrap the input. But I think it might not work because we wouldn't have the keyboard a11y for it (e.g. would still need to be able to arrow to the input). It would also pass the a11y checks but would be an issue for keyboard users since they are treated as separate tab stops.

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

Labels

dev-app preview When applied, previews of the dev-app are deployed to Firebase 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.

bug(mat-chip-grid): when used with a matChipInput, element has children which are not allowed

2 participants