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

Tiny things #8

Open
1 of 4 tasks
davelab6 opened this issue Jun 24, 2015 · 2 comments
Open
1 of 4 tasks

Tiny things #8

davelab6 opened this issue Jun 24, 2015 · 2 comments

Comments

@davelab6
Copy link
Contributor

  • the final binary has an FFTM table, which can be removed with fontTools but sadly it seems FontForge doesn't have an option to prevent it being included (Generate: Don't include FFTM fontforge/fontforge#2389)
  • the name table includes items with PlatformID="1" but these can be removed with fontTools
  • the name table version string (item 5) had a trailing space
  • the canonical filename of the font should be DigitalNumbers-Regular.ttf

I fixed these things in the version I committed to google/fonts@e860dd8 - please use this in the dist/ directory, and cut a release on https://github.com/s-a/digital-numbers-font/releases :)

s-a referenced this issue in google/fonts Jun 25, 2015
@s-a s-a mentioned this issue Jun 25, 2015
@s-a
Copy link
Owner

s-a commented Jun 25, 2015

@davelab6 not easy to hit your requirements 😄
Can you post us the scripts so we can fix these font tools stuff within the build job?
I have researched the name table version string, but found nothing within fontForge. You do not mean the font file version string?

@davelab6
Copy link
Contributor Author

The fontTools stuff should be in https://github.com/googlefonts/ although its buggy

I have researched the name table version string, but found nothing within fontForge. You do not mean the font file version string?

Maybe a nit in my process, never mind :)

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

No branches or pull requests

2 participants