-
Notifications
You must be signed in to change notification settings - Fork 83
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
fix: set input value font weight property on input container #8141
Conversation
Quality Gate passedIssues Measures |
aac2dcb
to
78e03eb
Compare
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.
In which way would this alter existing behavior? It preserves the defaults, and it looks like you would still be able to override everything in custom styles.
Yes, the only thing is that incorrect default (400) was changed to the one actually used. So maybe it's ok to consider this a regular fix. |
This ticket/PR has been released with Vaadin 24.6.0.alpha4 and is also targeting the upcoming stable 24.6.0 version. |
Description
Fixes #7957
Moved the font-weight custom CSS property to the correct file. Also changed it to
500
for backwards compatibility in order to not break screenshots (currently used400
has never been working properly).Marked as a behavior altering fix, though I think we can still backport it at least to 24.5 to make the property work.
Type of change