excalidraw / virgil

The old-font that powered Excalidraw. Now replaced by Excalifont.
https://plus.excalidraw.com/virgil
SIL Open Font License 1.1
624 stars 18 forks source link

fixes to the position and size of the t, number 1, and the slash #58

Closed wietsevenema closed 3 years ago

wietsevenema commented 3 years ago
Before After
Screenshot 2021-05-24 at 11 42 14 Screenshot 2021-05-24 at 11 41 11

This PR fixes #55 and #57

vercel[bot] commented 3 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/excalidraw/virgil/8yaKJkzSEEfrU1uwq1fPzpzyEfZP
✅ Preview: https://virgil-git-fork-wietsevenema-minor-fixes-excalidraw.vercel.app

thorn0 commented 3 years ago

Please run yarn fix

dwelle commented 3 years ago

What's the workflow once the PR is merged for the source file to be exported to .woff2?

thorn0 commented 3 years ago

There is no established workflow yet. So far we found only one way to convert vfj to woff2, namely export in FontLab. We can ask @wietsevenema to export the font to woff2 and add the result to this PR.

wietsevenema commented 3 years ago

✅ Added the export

wietsevenema commented 3 years ago

Let me know if something is still blocking this from merging, I'm ready to make changes while my Fontlab trial lasts :)

dwelle commented 3 years ago

I'll defer to @thorn0. If everything's fine we can merge and deploy tomorrow!

dwelle commented 3 years ago

I'm merging. If there are any issues we can fix in future PRs.

Thanks @wietsevenema! ❤️