-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Prevent clipping input text in Chrome + Segoe UI Font #14179
Prevent clipping input text in Chrome + Segoe UI Font #14179
Conversation
Can't reproduce on macOS but I wonder what's causing the clipping in first place. Normally, such clipping is caused by |
Perhaps if you use Segoe UI font (which is what I get with
Hmm, appears your right. I assumed that since left/right padding clip text that the bottom padding would clip as well. However, when I set I'll update the Chrome bug. I'm okay if you want to close this and just wait for a fix to come in Chrome. I'll probably merge the change in my fork. |
@silverwind I've seen clipping like this too. - |
https://try.gitea.io/arandomer/merge-test with noto sans |
this pr does fix that issue |
Codecov Report
@@ Coverage Diff @@
## master #14179 +/- ##
==========================================
- Coverage 42.02% 42.00% -0.03%
==========================================
Files 734 734
Lines 78780 78780
==========================================
- Hits 33106 33088 -18
- Misses 40239 40251 +12
- Partials 5435 5441 +6
Continue to review full report at Codecov.
|
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.
Please update with base branch
Closes #14162
Before
After
The overall element height stays the same.
There is an issue in Chrome + Windows where the line-height is cropped too short. It seems like at least partially an issue with how Chrome calculates the height, but this fix should be more stable for different user fonts.
Note: I changed only the clone panel css classes. The other input elements remain as they were, as they didn't have the same issue with cropping the text.