-
Notifications
You must be signed in to change notification settings - Fork 103
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
New fontbakery-fix-ttfautohint.py script #665
Comments
Feel free to grab and use anything that proves useful from font-ttfa or let me know if you'd like me to submit a PR here rather than in https://github.com/twardoch/fonttools-utils as you suggested in the issue report. Here's the start of the meat of the script https://github.com/source-foundry/font-ttfa/blob/master/lib/fontttfa/app.py#L58-73 |
Thanks!! Fontbakery is a bit bloated and this update tool is a bit different to yours, so adams' repo is better, I think :) |
👍 |
The 0.1.0 rewrite should do this |
@felipesanches what is the status of this? |
994ea77 introduces a script that will print the table information if present. The next action is to implement the autofix method as described above. This is a 'misc' category task with relatively low importance. |
These days we don't fix fonts in fontbakery, but we already have a check for making sure that ttfautohint is up to date. Feel free to reopen if I've misunderstood. |
A lot of fonts in the gf collection have old versions of ttfautohint applied, and it would be great to have a script that initially uses the methods in
fontbakery-fix-version.py
that...-I
or-t
args respectively were passed to ttfautohint, to record its args in the ttf file) (there is a pypi package https://pypi.python.org/pypi/font-ttfa for reading the ttfa table, although per Such a big repo for such a small tool :) source-foundry/font-ttfa#1 it might be better to inline the code... :)--autofix
arg/method that runs the latest version of ttfautohint using the options from the name table.I encountered this while updating Raleway, impallari/Raleway#2
The text was updated successfully, but these errors were encountered: