-
Notifications
You must be signed in to change notification settings - Fork 2
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
For Braille script (Brai), add font Noto Sans Symbols 2 #14
base: main
Are you sure you want to change the base?
Conversation
Here's the whole block:
Noto Sans Symbols 2 support: Catrinity's support from their Overview (all visible characters without further information, PDF): |
I could add Catrinity in the "Other" column, but it seems unlikely to me that the default font should include the open dots (and possibly not have different 6- and 8- dot glyphs). |
@LornaSIL Do you think Catrinity should be added here and to https://github.com/silnrsi/fonts/blob/main/families.json? |
They both have the same character set (for the Braille block). As you say, it's odd to have the open cells, so just stick with the Noto font. |
@mhosken This PR is ready to merge, but I know you're doing some debugging and I don't want to trigger another build until you're ready. Feel free to merge this whenever it's convenient for you (or let me know and I'll do it). |
@imnasnainaec Thanks for this contribution! |
Per https://fonts.google.com/noto/use#how-are-noto-fonts-organized, penultimate paragraph in "Noto families":