bug(chips): matChipRemove should not apply to icons

[jelbourn]

All of our examples that use matChipRemove apply directly to mat-icon, which is not great for a11y; instead, the directive should go on a button that contains an icon.

Alternatively, we can project <mat-icon matChipRemove> into a button within the component. This fix might be preferable since it will "fix" anyone who has copied this example so far.

[crisbeto]

I think that we'll have to make some changes before we recommend <button matChipRemove>, because the user agent styles will interfere with the icon.

[jelbourn]

@crisbeto yeah, definitely. We could try making button[matChipRemove] a component with the right styles, but I don't have a sense off the top of my head how breaking that would be.

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}