-
Notifications
You must be signed in to change notification settings - Fork 56
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
Increase 1px margin for form messages to conform to design #1948
Conversation
✅ Deploy Preview for boosted ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
# Conflicts: # scss/_variables.scss
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
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.
LGTM
All looks good. No issues. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Note: Please transform
- [ ]
into- (NA)
in the description when things are not applicableRelated issues
Linked to issues #1761 and #1767.
Based on the discussions that happened in #1779 (direct link).
Description
During the changes made on form texts to conform to new UI kit V5, it has been seen that the margin between form inputs and messages (both form texts and validation messages) was lacking 1px.
Motivation & Context
This change is required to have the exact same height between the bottom of the input field and the top of the uppercase letters at 10px, both in Boosted and in designs.
Types of change
Live previews
Checklist
Contribution
Accessibility
Design
Development
Documentation
Checklist (for Core Team only)
After the merge