-
Notifications
You must be signed in to change notification settings - Fork 4.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Navigation component: Add color to menu and group title #25507
Conversation
This makes it more specific so it won't be overriden by other CSS
Size Change: -1 B Total Size: 1.2 MB
ℹ️ View Unchanged
|
@david-szabo97 I'd suggest adding these colors to the original PR #25506 instead, and close this. At first I was about to ask you to replace the hex codes with the G2 color, but then I realized we already use that same hex color (and others) elsewhere in the Nav styles. Eventually we'll need to discuss, revamp, and polish the menu colors, so imho this change is not essential, even more so as #25506 is currently an exploration. 🙂 |
@Copons Eventually we will need to add a color to these elements. Since they aren't a simple div or span which is usually unstyled. I'm fine with adding this little change in the exploration PR, but I'd rather have it merged to make sure we don't forget about it. I'm sure the color will change in the future when the design is finalized, but it's easier to find and replace than figure out why it isn't working as expected. |
Description
Add
color
property toMenuTitleUi
andGroupTitleUi
to make them more specific.See #25506
Types of changes
Bug Fix
Checklist: