KEMBAR78
feat: aura accent color for button, side-nav, radio-button and checkbox by jouni · Pull Request #10249 · vaadin/web-components · GitHub
Skip to content

Conversation

@jouni
Copy link
Member

@jouni jouni commented Sep 28, 2025

Allow developers to customize --aura-accent-light and --aura-accent-dark on a component level for specific components.

Define --aura-accent-contrast-light and --aura-accent-contrast-dark explicitly, in addition to the compound --aura-accent-contrast property.

Rename the Side Nave contrast variant to filled.

Includes a few unrelated fixes to Grid theme as well (use the property padding custom property, empty state styles).

@jouni jouni mentioned this pull request Sep 29, 2025
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']) {
Copy link
Member

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?

Copy link
Member Author

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.

Copy link
Member

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,
Copy link
Member

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.

@sonarqubecloud
Copy link

@web-padawan web-padawan merged commit f6571da into main Sep 29, 2025
9 checks passed
@web-padawan web-padawan deleted the feat/aura-component-accent branch September 29, 2025 11:55
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Vaadin 25.0.0-alpha14.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants