fix(MenuItem): icon only active state in Teams theme

[mnajdova]

Fix #1465

Menu with icon only, didn't show any active state.

Teams default theme

Before:

After:

Teams dark theme

Before:

After:

Teams high-contrast theme

Before:

After:

Question

Not sure whether the icon in hc should be outline/filled. On focus it is filled, so I added the same logic here. I may revert this changes.

[codecov]

Codecov Report

Merging #1464 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master    #1464   +/-   ##
=======================================
  Coverage   73.14%   73.14%           
=======================================
  Files         805      805           
  Lines        6073     6073           
  Branches     1775     1794   +19     
=======================================
  Hits         4442     4442           
  Misses       1625     1625           
  Partials        6        6

Impacted Files	Coverage Δ
...ams-high-contrast/components/Menu/menuVariables.ts	0% <ø> (ø)	⬆️
...src/themes/teams/components/Menu/menuItemStyles.ts	7.2% <0%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 47dca77...36312ba. Read the comment docs.

[kuzhelov]

just to be sure - could you, please, suggest where original inherit value had resulted in suboptimal styles?

[mnajdova]

In team's theme the iconOnly Menu should always have the brand color for the active items. The inherit color is black in default theme.

[layershifter]

startItem & endItem?

[kuzhelov]

@codepretty, could you, please, suggest what is prescribed by Teams design spec for the case @mnajdova had mentioned in the description? Thank you!

Not sure whether the icon in HC should be outline/filled. On focus it is filled, so I added the same logic here. I may revert this changes.