add theme override: menu item selected + hover

[wc-matteo]

This function solves the problem (这个功能解决的问题)

To style the text/icon/etc when a menu item is selected differently than when it's not selected while the user is hovering on that item

Expected API (期望的 API)

add new theme override props like: itemTextColorActiveHover
(don't forget the inverted variant!)

[Talljack]

The design is originally designed like this, which is more convenient for users to adjust. If you need to keep the hover and active colors the same, you can set these two variables to be the same.

[wc-matteo]

unless I'm mistaken, right now there is now way (using theme override) to have a different hover text/icon/etc color ONLY for the selected menu item:

to give an example this is what I implemented using n-menu + custom css:

when the menu item is selected the hover text/icon/etc should be a dark color (cause the selected menu item bg is light), when it's not selected, the opposite (cause the menu item bg is dark).

BONUS

would be nice to have a configurable hover bg like this as well

[Talljack]

Sorry, we understand the instance you said, but you set the itemColorActive variable to set the bg, and then you can set the color of the hover through the itemTextColorHover and itemIconColorHover variables, the effect is as above.

[wc-matteo]

yes but I need 2 different hover colors! I need a hover color for when I have the active bg and another hover color for when I don't have the active bg.

[Talljack]

yeah, maybe add new theme override props

[07akioni]

yes but I need 2 different hover colors! I need a hover color for when I have the active bg and another hover color for when I don't have the active bg.

This is really a tough problem. How to define the theme variable name. We need some time to think.

[wc-matteo]

there already is itemXXXColorActive and itemXXXColorHover. You could add itemXXXColorActiveHover.

Also. it'd be nice to have the term actve be changed to selected. To better align it with the term used in the documentation and because active also represents another concept.

[07akioni]

there already is itemXXXColorActive and itemXXXColorHover. You could add itemXXXColorActiveHover.

Also. it'd be nice to have the term actve be changed to selected. To better align it with the term used in the documentation and because active also represents another concept.

Active in naive ui is named as pressed.