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(NcInputField): Adjust styling of the internal label #4578
Conversation
066ff1f
to
64aa149
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.
Nice! Much better already!
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.
Much better! But shouldn’t we use variables in all those places, based on e.g. var(--default-grid-baseline)
?
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.
I'd go for 4px border-radius, not pill
* Add note that the internal label is only recommended when the background matches * Make border of the internal label rounded (needed to adjust the padding as well) * Perfect pixel align label, placeholder and content Co-authored-by: Marco <marcoambrosini@icloud.com> Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
e20ecde
to
b4b8860
Compare
☑️ Resolves
From today's design review call:
🖼️ Screenshots
(added background color to make the border more visible, e.g. like on the login)
🏁 Checklist