Skip to content

experiment: add select base styles and visual tests #9189

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

web-padawan
Copy link
Member

@web-padawan web-padawan commented May 16, 2025

Description

Added vaadin-select base styles and visual tests from the base-styles branch.
This also includes base styles for vaadin-overlay which will be used by other components too.

Note: I decided to preserve vaadin-overlay-styles.js file to avoid changing imports for it everywhere.
The actual overlayStyles that it provides will still be correctly resolved to base styles in visual tests.

UPD: two adjustments for vaadin-select-value-button styles:

  • Added pointer-events: none for disabled state to avoid focusing select when it is disabled.

Type of change

  • Experiment

@web-padawan web-padawan requested review from jouni and vursen May 16, 2025 15:03
@web-padawan web-padawan force-pushed the experiment/select-base-styles branch 2 times, most recently from 151e746 to 2b80e86 Compare May 16, 2025 15:09
@web-padawan web-padawan force-pushed the experiment/select-base-styles branch from 2b80e86 to 03d76ff Compare May 16, 2025 15:16
@web-padawan
Copy link
Member Author

Will update to use cursor custom CSS properties once #9190 is merged.

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.

1 participant