X Tutup
The Wayback Machine - https://web.archive.org/web/20200704141558/https://github.com/microsoft/cascadia-code/pull/260
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

Updating scripts for OTF and WOFF2 #260

Merged
merged 17 commits into from May 5, 2020
Merged

Updating scripts for OTF and WOFF2 #260

merged 17 commits into from May 5, 2020

Conversation

@aaronbell
Copy link
Collaborator

aaronbell commented May 2, 2020

Summary of the Pull Request

Updated the build.py script to also build OTF versions of the Cascadia Code fonts (in light of the blue zones added!). Additionally added a "build_WOFF.py" script that will process the TTF files in the "build" folder to produce WOFF2 files. Throws an error if VTT production tables present in the fonts.

PR Checklist

  • Closes #236
  • CLA signed.

Detailed Description of the Pull Request / Additional comments

  • Updated build.py to also build OTF file
  • Added build_WOFF.py to process TTFs to create WOFF2 files (if VTT production files have been removed)
  • Updated the Requirements.txt file to include compression library for WOFF2 generation
Aaron and others added 11 commits Dec 4, 2019
Aaron
Adding a few missing glyphs, modifying PL characters for better alignment and resolving issue with Apple's ATSUI API.
This reverts commit ade4ef1.
Aaron
Updating the font to resolve a number of open bugs. Also, cleaned up a number of glyphs.
Aaron
Offsets for subscripts and inferior numbers are set inconsistently. These are now corrected.
Aaron
Modified the build.py script to add in OTF generation, and also created a second build script that created WOFF2 files from the TTF files (if TSI data removed). Requirements.txt updated to include WOFF2 compression library.
@aaronbell aaronbell requested a review from DHowett-MSFT May 2, 2020
Copy link

DHowett-MSFT left a comment

This looks great!

build.py Outdated Show resolved Hide resolved
@aaronbell
Copy link
Collaborator Author

aaronbell commented May 2, 2020

@DHowett-MSFT I figured you can add the WOFF2 converter to your build script to make things easier for ya :D

Co-authored-by: Nikolaus Waxweiler <madigens@gmail.com>
@aaronbell aaronbell marked this pull request as ready for review May 2, 2020
build.py Show resolved Hide resolved
build_WOFF.py Outdated Show resolved Hide resolved
@madig
Copy link
Contributor

madig commented May 2, 2020

You may also want to run psautohint over the OTFs like in https://gitlab.gnome.org/GNOME/cantarell-fonts/-/blob/master/scripts/make-static-fonts.py#L38-39

Aaron added 2 commits May 2, 2020
Not sure how these got back in. Removing again.
Aaron
@aaronbell aaronbell requested a review from DHowett-MSFT May 2, 2020
@aaronbell
Copy link
Collaborator Author

aaronbell commented May 2, 2020

Added psautohint functionality to the build script and also tried again to clean out the hints from those imported glyphs. I thought I'd gotten them before, but somehow snuck back.

@aaronbell
Copy link
Collaborator Author

aaronbell commented May 2, 2020

BTW, psautohint was kicking out a whole slew of warnings about 'bogus BBox" I assume partly because of the oddity of the ligatures. Anyway, I filed those away to a "log.txt" file in the build folder.

build.py Outdated Show resolved Hide resolved
@madig
Copy link
Contributor

madig commented May 3, 2020

I think psautohint is a bit anal about that, but I don't know the real value of those warnings.

@aaronbell
Copy link
Collaborator Author

aaronbell commented May 3, 2020

It seemed to me that the hinting all went through despite the warnings—so I figured they were just for reference rather than any specific blocker.

Aaron
@aaronbell
Copy link
Collaborator Author

aaronbell commented May 4, 2020

Derp. I totally forgot that it doesn't just automerge the suggested changes. Alright, I've gone through and fixed those up :)

build_WOFF.py Show resolved Hide resolved
Aaron added 2 commits May 4, 2020
Aaron
Aaron
@aaronbell aaronbell merged commit 4bd34ed into microsoft:master May 5, 2020
1 check passed
1 check passed
license/cla All CLA requirements met.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

3 participants
You can’t perform that action at this time.
X Tutup