feat(Tag): add pointer on hover if component is interactive

[emyarod]

Closes #7592

This PR adds a pointer cursor on hover when an onClick handler is supplied to the Tag component

Testing / Reviewing

Ensure that the interactive tag example changes cursor on hover

[netlify]

Deploy preview for carbon-elements ready!

Built with commit 6922f53

https://deploy-preview-7839--carbon-elements.netlify.app

[netlify]

Deploy preview for carbon-components-react ready!

Built with commit 6922f53

https://deploy-preview-7839--carbon-components-react.netlify.app

[jnm2377]

Should we also add a bg hover color, and maybe focus state for interactive as well? Since it'll be clickable/keyboard accessible. We have a bg hover color for the x in the filterable tag, I would expect it to be the same color for interactive tag.

[emyarod]

this is only restoring a reported regression in pointer styles (the tags remain keyboard inaccessible, keeping the current behavior). I don't have the context from the Slack conversation referenced in the original ticket but from what I understand, those style changes have not been specced out yet. but if we want to roll in the regression fix and the interactive style updates together I will need a spec from design. is that how we should proceed?

[tw15egan]

cc @aagonzales

#7592 (comment)

[emyarod]

yeah I reached out to her about this and I'm watching the thread. not sure if this means this regression fix needs to be blocked

[tw15egan]

@emyarod Looks like @aagonzales has given this the go-ahead. Hover looks good, but doesn't seem to be focusable via keyboard. Should we forward any onClick handlers to an internal onKeyDown / add a tabindex?

[emyarod]

yeah I'm currently discussing potential hover/active states for interactive tags with her. this PR was initially to restore the reported regression for hover state but depending on the discussions then more style and functionality will need to be introduced

[aagonzales]

Perfect, I think the hovers are a nice addition. Thanks for adding those!

[jnm2377]

Looks great! 🔥

[tw15egan]

LGTM 👍 ✅