Skip to content
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

very large margin for the first line #78

Open
GamingKing2436 opened this issue Feb 23, 2024 · 3 comments
Open

very large margin for the first line #78

GamingKing2436 opened this issue Feb 23, 2024 · 3 comments

Comments

@GamingKing2436
Copy link

for fairfaxHD, it has a very large margin on the first line for some reason. this did not happen in a december version of the font. it also does not apply to the SMHD version.
see example in this font viewing app:
Screenshot_20240222_214500_FontViewer
used as a font in other apps:
Screenshot_20240221_085517_Settings
not sure what causes this yet.

@GamingKing2436
Copy link
Author

okay SMHD does have a large margin for the first line, but it's only a small subtle increase, which means all the fonts were probably intended to have some small margin. the other fonts be having like 3x margin tho.

@RebeccaRGB
Copy link
Member

Android is doing something very stupid here and using the font bounding box to determine the line height of the first and last line instead of the ascent, descent, and line gap metrics that it should be using.

There are some recent changes that attempt to fix this but I don't know when they'll be released / if they're already released / if they actually fix this problem / if they actually fix this problem in every case or just in some cases / etc.

I may have to provide an ugly hacked version just for Android. :/

@GamingKing2436
Copy link
Author

lol funny hack

i'm pretty sure that would be only in some cases, as apps can apperently choose their own includeFontPadding value (probably rare tho). it's been released, but stable 1.6.0 was only released on January 24, 2024, so it'll probably take a while anyways.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants