-
-
Notifications
You must be signed in to change notification settings - Fork 78.9k
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 pseudo paddings and height #30269
Conversation
Holy shit, is this really the cause of all that pain? No requirement for fixed heights on form controls now? :D |
On iOS (Chrome/Safari) there still seems to be a problem. |
FYI, this patch is unrelated to #30353.
Not sure about that, we got to check that in #30353. We added |
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
It turns out the height of date inputs (#18842) are increased by the padding on the pseudo elements and the fixed height of
1.5rem
of the::-webkit-inner-spin-button
.I've now reset these paddings and height instead of hacking with the line-height.
Codepen to play with: https://codepen.io/MartijnCuppens/pen/wvBLzvb?editors=1100