Fix #7082 multiple hover styles

[ixkaito]

Description

Fix the issue (#7028) that block toolbar buttons use multiple hover styles.

How has this been tested?

Ran npm test:

Test Suites: 1 skipped, 178 passed, 178 of 179 total
Tests:       3 skipped, 1598 passed, 1601 total
Snapshots:   59 passed, 59 total
Time:        54.724s
Ran all test suites.

Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist:

• My code is tested.
• My code follows the WordPress code style.
• My code follows the accessibility standards.
• My code has proper inline documentation.

[aduth]

I believe the issue meant to be addressed is #7082 (the original comment mentions #7028).

[ixkaito]

@aduth Thanks! You're right. It's #7082 .

[jasmussen]

Awesome, thanks so much for finding this issue and taking a stab at fixing it. Really appreciate it 🏅

There was an issue with your approach, though, which meant that hover styles were broken for other icon buttons as a result of this. For example this button needs the hover style supplied by the box shadow:

I took the liberty of pushing a tweak to your code, which instead hides the hover style on the formatting button itself instead of in the icon button component:

[jasmussen]

Works well now!

I'll let @aduth take a quick look and merge if it's 👍 👍

[ixkaito]

@jasmussen Thank you for your follow-up!

[jasmussen]

Merging as this would be good to get in master.