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

v3.000: Support variable fonts #43

Merged
merged 37 commits into from
Nov 30, 2018
Merged

v3.000: Support variable fonts #43

merged 37 commits into from
Nov 30, 2018

Conversation

m4rc1e
Copy link
Collaborator

@m4rc1e m4rc1e commented Nov 20, 2018

I've been working on this for the past two months. Micah, Eli and Mike have been using it frequently so I think we've found most of the major issues.

Apart from adding VF support, the following improvements have been made:

Atm, it can't use a font's width axis (same as GF). I hope to have this in place once we know what values to use, google/fonts#1706

I'm also not updating www.gf-regression.com until I have the following in place:

  • Logging/prompts for users.
  • Progress bars (producing diffs for large families can be time consuming, users may believe the app has crashed)

Even by making these updates, the hardware it currently runs on isn't sufficient anymore. I would prefer people to use fontdiffenator instead, especially since it will be able to generate diff images soon. My current setup is to use the new GFR instance with diffbrowsers and fontdiffenator.

The previous implementation used the font's nametable.
This approach was too unreliable.
Sometimes diffing the families isn't needed. A user may just want to
compare two fonts in a waterfall (for hinting purposes) or play with
the editor.
Hinters want to see all the glyphs contained in a font
User will have to provide an Access-Token in header to post media
@davelab6
Copy link
Member

davelab6 commented Nov 22, 2018

@m4rc1e ready to merge?

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

Successfully merging this pull request may close these issues.

None yet

2 participants