-
Notifications
You must be signed in to change notification settings - Fork 96
Font Suggestion: Source Han Sans #108
Comments
Thanks! There's a huge backlog of fonts already; I don't think I'll be able to start working on adding fonts until my exams are done (start of May or start of June). |
I could help add fonts if you want via pull request. |
@TriforceOfKirby It is a bit tricky, but if you want to tackle it, that would be great! |
@alfredxing Ok so I opened the font in FontForge, then went to It gives me a warning message:
If I click Clicking I'm not sure if this is right, I don't think the font file size should shrink that much. |
@TriforceOfKirby Yeah, I think it's a problem with this being an Asian font, which might have different data than regular fonts... I haven't worked with CID fonts before, but maybe try flattening them? This page provides docs on manipulating CID fonts in FontForge. |
@alfredxing So flattening seems to work, but I do get a couple warning messages:
I still get the font file, but I'm not sure if it was created correctly. I made a test web page with the font embedded on Dropbox. |
That should be okay. It'll just be one character probably, and there's not much we can do other than that. Did the em-size error go away? Thanks for doing this, by the way. It really helps out! |
@alfredxing No, the em-size warning still shows. Although, this page on FontForge's website says that 1000 is a perfectly fine em size. You're welcome! I'll help out with a few more fonts too. |
Yep, 1000 is fine. I think (though I'm not sure) that 1000 is used for PostScript outlines, and a power of two (like 1024 or 2048) is usually used for TrueType. |
@alfredxing So I found a command-line tool by Mozilla for converting sfnt fonts(otf, ttf, etc.) to woff format here. With that I ran this batch command: |
https://github.com/adobe-fonts/source-han-sans
The text was updated successfully, but these errors were encountered: