coz-m / MPLUS_FONTS

M+ FONTS
https://mplusfonts.github.io
SIL Open Font License 1.1
903 stars 61 forks source link

Finalizing build process #5

Closed aaronbell closed 3 years ago

aaronbell commented 3 years ago

The build process has been generally finalized for the Mplus font families. At current, 33 fonts (!) are being made automatically:

Mplus 1 - variable font & 7 static instances Mplus 2 - variable font & 7 static instances Mplus Code - variable font & 5 static instances Mplus Latin - variable font & 10 static instances

I made some minor modifications to the fonts where I caught issues or to update metadata but otherwise kept the sources as is.

If you find any issues, or have any other concerns, please let me know!

aaronbell commented 3 years ago

@mintoming Just updating you :)

coz-m commented 3 years ago

Thank you for helping with the very difficult task.

Is it possible to change some font names back?

There is a possibility of a bug in Illustrator, but some glyphs are broken when displaying variable fonts.

aaronbell commented 3 years ago

Thanks. I'll correct the names.

I wasn't able to repro the Mplus1 and Mplus2 bugs in Illustrator. No idea what would be causing those.

The bug in Mplus Latin is odd. I think it is Illustrator related as well, but I can resolve it by removing the overlaps in the corner.

coz-m commented 3 years ago

Thanks. I'll correct the names.

I'm sorry for granting my selfishness.

I wasn't able to repro the Mplus1 and Mplus2 bugs in Illustrator. No idea what would be causing those.

Roger that. It's a mysterious phenomenon.

The bug in Mplus Latin is odd. I think it is Illustrator related as well, but I can resolve it by removing the overlaps in the corner.

https://user-images.githubusercontent.com/55127265/117790327-36380400-b284-11eb-82b6-65579bd74de9.mov

Correct behavior with Mplus 1.

https://user-images.githubusercontent.com/55127265/117790552-6e3f4700-b284-11eb-82f2-7bac5bfa2586.mov

I'm sorry for the detailed request, but please consider it.

coz-m commented 3 years ago

Excuse me for the addition, can we publish OTF along with TTF on GitHub?

aaronbell commented 3 years ago

I've implemented all the requested changes bar 2: 1) Unfortunately, per my understanding of Google's naming requirements, variable fonts must align with certain naming conventions and those conventions mean that the Latin can't have specific names like "Thin 50" or such for the instances.

We only allow weight and italic particles. If a font contains additional axes, they must not be mentioned in the instance names

2) We certainly can create OTFs for the repro, though they won't get used on Google Fonts. I have not built them at current as it'll extend my build time, and I want to make sure you're happy with the fonts first :)

coz-m commented 3 years ago

Thank you for responding to my many requests. I regret the bug of displaying variable fonts in Illustrator, but I am very happy that you made them variable fonts including kanji glyphs. I'm still incredibly surprised.

Please take the work to the next stage. And finally, please also create a OTF.

aaronbell commented 3 years ago

Thank you!

BTW, I happened to try out Mplus 1 on my laptop that has an older version of Illustrator installed (23.0.1) and was able to repro the bug. However, after upgrading to the latest version of Illustrator (25.2.3), the bug disappeared.

As such, I believe that this bug is definitely related to Illustrator.

coz-m commented 3 years ago

Thank you for your confirmation. I also confirmed that the bug was fixed by upgrading from 24.0.1 to 25.2.3. I was relieved :-)