-
Notifications
You must be signed in to change notification settings - Fork 87
feat: aura accent color for button, side-nav, radio-button and checkbox #10249
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
Conversation
| vaadin-side-nav:not(:has([slot='label']))::part(label) { | ||
| display: none; | ||
| /* Badges and other content that uses the palette inside a filled nav item */ | ||
| vaadin-side-nav[theme~='filled'] vaadin-side-nav-item[current] > :not([slot='children']) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we change selector to ensure this doesn't affect tooltips in slot="tooltip" added in #10008?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, should exclude tooltips as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated.
| :where(:root, :host), | ||
| vaadin-button, | ||
| vaadin-menu-bar-button, | ||
| vaadin-drawer-toggle, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not a blocker for this PR, just a general observation: we have some lists of components both in color.css and also surface.css, there can be presumably more of these?
While overall the code is quite self-explanatory, it would not harm to write down some short clarifications behind these at least in comments (they are minified anyways), some sort of file overview etc.
|
|
This ticket/PR has been released with Vaadin 25.0.0-alpha14. |



Allow developers to customize
--aura-accent-lightand--aura-accent-darkon a component level for specific components.Define
--aura-accent-contrast-lightand--aura-accent-contrast-darkexplicitly, in addition to the compound--aura-accent-contrastproperty.Rename the Side Nave
contrastvariant tofilled.Includes a few unrelated fixes to Grid theme as well (use the property padding custom property, empty state styles).