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

Better target_bb selection #3

Open
ahti opened this issue Jul 15, 2014 · 0 comments
Open

Better target_bb selection #3

ahti opened this issue Jul 15, 2014 · 0 comments

Comments

@ahti
Copy link

ahti commented Jul 15, 2014

I have found that for the Menlo font, the default way to determine the target_bb gives very bad results (the box drawing characters extend beyond the normal character bounds quite a bit).

Due to this, every patched Menlo i found online looked very bad (at least to me, most others seem not to notice 😅).

I have now found a target_bb that looks good to me (at least at 288pt font size and below).

Now that I have gone through this work, I think it would be a shame to just use it for me. But I also dislike the idea of me just publishing this patched Menlo online somewhere. I think a more generic approach would be useful.

Thus I think it would be a good idea to include target bounding boxes for specific fonts in fontpatcher, and fall back to the default way of going through characters if there is no bounding box for the font to be patched.

This way, everyone who uses the patcher with Menlo in the future would get the better target bounding box.

I am not very well versed in python, so I don't know what the best way to keep this data separate from the actual script would be, or I would have set up a pull request, but if you like this idea, here is the target_bb that I have found works well for Menlo: [0, -483 1227, 1936]

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

1 participant