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

add script generate-fonts #131

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Download
Copy link

@Download Download commented Jan 7, 2024

This PR adds a folder generate-fonts under scripts/ that contains a small Node JS project that generates ttf and woff2 fonts for the assets in this project.

SEE #16

@Download
Copy link
Author

Download commented Jan 7, 2024

@microsoft-github-policy-service agree

@Download
Copy link
Author

Download commented Jan 7, 2024

It does not generate font files for the 3D variant because there are no .svg files for that variant.

It also does not generate font files for the Color variant, because when I try to do so I encounter an error:

Error: Got an error parsing the glyph "alien": Unexpected character "N" at index 1. Command cannot follow comma.

Alien is not the first glyph, so in principle Color can work, but I cannot easily make it skip a glyph in case of errors. I am not sure whether the alien glyph is an exception or whether there are many glyphs with this problem in the Color set. That would need some further investigation.

@ahydronous
Copy link

@willchavez seeing as you're still active in this repo, could you merge this or contribute Microsoft's own build scripts?

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.

2 participants