Skip to content

Conversation

@web-padawan
Copy link
Member

@web-padawan web-padawan commented Dec 19, 2025

  1. Added the following custom CSS properties that were missing:
  • --vaadin-checkbox-border-width
  • --vaadin-checkbox-gap
  • --vaadin-checkbox-label-font-weight
  • --vaadin-checkbox-label-line-height
  • --vaadin-radio-button-gap
  • --vaadin-radio-button-label-font-weight
  • --vaadin-radio-button-label-line-height
  1. Modified some properties as supported by both Lumo and Aura:
  • --vaadin-checkbox-border-radius
  • --vaadin-checkbox-label-color
  • --vaadin-checkbox-label-font-size
  • --vaadin-radio-button-dot-color
  • --vaadin-radio-button-dot-size
  • --vaadin-radio-button-label-color
  • --vaadin-radio-button-label-font-size
  1. Sorted the vaadin-checkbox style properties table alphabetically.

Comment on lines 68 to 72
|`--vaadin-radio-button-dot-color`
|Lumo
|Aura, Lumo

|`--vaadin-radio-button-dot-size`
|Lumo
|Aura, Lumo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the idea here was that you should use the newer --vaadin-radio-button-marker-color / --vaadin-radio-button-marker-size props, which are aligned with the naming in checkbox. However those doesn't work for Lumo, so the dot properties are listed as well.

Maybe we could update the props with the old name in checkbox as well, to indicate you only should use them with Lumo.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point, thanks. Restored these to mention Lumo and updated old checkbox props accordingly.

@web-padawan web-padawan force-pushed the docs/checkbox-radio-props branch from 89102ff to 52fc95b Compare December 22, 2025 07:57
@sissbruecker sissbruecker merged commit f84f20a into main Dec 22, 2025
3 checks passed
@sissbruecker sissbruecker deleted the docs/checkbox-radio-props branch December 22, 2025 10:29
@web-padawan web-padawan removed the request for review from vursen December 22, 2025 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants