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

Return c_t, s_k, s_p, and s_t to hlig from dlig #32

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

LiberalArtist
Copy link

This PR returns the ligatures c_t, s_k, s_p, and s_t to hlig from dlig, restoring the categorization from @georgd's specimen.

Related to #20 and #4 (comment).

I expect some changes will be needed before this can be merged:

  • I don't have the Glyphs application. I used glyphs2ufo to generate .ufo and .designspace files, edited the features.fea files, and generated OTF and TTF formats using fontmake. If you want to keep .glyphs as the canonical format, presumably I should use ufo2glyphs to update the .glyphs files. However, I think it would make more sense to keep UFO in this repository, rather than a proprietary format.
  • I have not made any change to the build script for variable fonts, which might be needed depending on the decision about UFO vs. Glyphs.
  • The generated OTF and TTF files use names based on EBGaramond12 rather than EBGaramond, and the Medium, SemiBold, and ExtraBold weights are missing. Hopefully this is just a matter of passing the right options to fontmake.

This change restores the categorization from @georgd's [specimen].

Related to <octaviopardo#20>.
See also <octaviopardo#4 (comment)>.

To do:

- The .glyphs files should be either updated or removed.

- The build script for variable fonts should be updated.

- Generated files changed to use names based on "EBGaramond12".
  They should probably go back to using "EBGaramond", instead.

- Medium, SemiBold, and ExtraBold weights are missing and
  should be restored.

[specimen]: https://github.com/georgd/EB-Garamond/blob/82c892d3b1ecf6fe52554c5133af9127974b9be8/specimen/Specimen.pdf
AndrewKvalheim added a commit to AndrewKvalheim/EBGaramond12 that referenced this pull request Dec 24, 2020
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.

1 participant