fix(overflow-menu): update trigger's hover color

[asudoh]

Refs #3210.

Changelog

Changed

• The hover color of overflow menu's trigger button.

Testing / Reviewing

Testing should make sure our overflow menu is not broken.

[netlify]

Deploy preview for the-carbon-components ready!

Built with commit 668d729

https://deploy-preview-4270--the-carbon-components.netlify.com

[netlify]

Deploy preview for carbon-components-react ready!

Built with commit 668d729

https://deploy-preview-4270--carbon-components-react.netlify.com

[netlify]

Deploy preview for carbon-elements failed.

Built with commit 668d729

https://app.netlify.com/sites/carbon-elements/deploys/5da9021b79a82f00080e40f7

[emyarod]

looks good to me, pending design review

[jnm2377]

I'll defer to design on this one. I am noticing that now, when you select a row and then hover on the overflow menu, there is no noticeable hover difference bc it's the same color as the row (that's what the $hover-selected-ui is for). So I'm not sure if overflow menu hover should be this color. I remember discussing this with @designertyler when I was working on data table but we didn't really discuss other options.

[designertyler]

We lose the overflow hover state on selected rows now that they are the same color. Is it possible to have a different overflow hover state for when the row is selected?

It's getting lost on selected rows since they are the same color.

[asudoh]

@designertyler Thank you for reviewing - Any ideas (e.g. choice of theme token) wrt what the hover color of overflow menu's trigger button on selected rows? CC @shixiedesign

[designertyler]

I'd suggest using the row's original selected color for the overflow menu's hover when it's on a selected row. This means the overflow menu's hover goes lighter for the 4 themes.

Designers chime in if this isn't what we want.

	White theme	Gray 10 theme	Gray 90 theme	Gray 100 theme
Selected row's overflow menu hover	gray-20 #e0e0e0	gray-20 #e0e0e0	gray-70 #525252	gray-80 #393939

[asudoh]

Thanks @designertyler for the updated spec! Fixed.