Update Tag to use Close icon

[taysea]

This ticket must wait until grommet/grommet#6870 is merged. Update Tag icon to use Close as opposed to FormClose.

[jcfilben]

Changing the tag icon to Close results in an icon that look unpropotionally large in comparison with the text in the tag.

Tag with Close icon

Tag with FormClose icon (this is what is currently implemented)

Pulling this issue back from this sprint. We need to look into the icon sizing more and determine what is the correct size to use with Tag.

[MikeKingdom]

Greg chose the FormClose icon when he originally designed the Tag partly for this reason.

Why do we currently feel it should be the bigger icon? Affecting the height of the Tag will cause alot of problems.

[jcfilben]

I think the desire is for the HPE theme to no longer use the 'Form' icons. But agree this causes issues with Tag. Seems like we need to have more design discussion here before moving forward with this.

[halocline]

I think the desire is for the HPE theme to no longer use the 'Form' icons. But agree this causes issues with Tag. Seems like we need to have more design discussion here before moving forward with this.

Or, theme does something like

icon: <Close />,
size: 'small'

Don't know if that is valid or not, but an avenue to consider.