-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Noto Serif: Version 2.014 added #8182
base: main
Are you sure you want to change the base?
Conversation
Taken from the upstream repo https://github.com/notofonts/latin-greek-cyrillic at commit notofonts/latin-greek-cyrillic@22328ea.
|
@simoncozens
but I realised that it has less changes (Noto Sans had a lot of modifications in devanagari, if you have the corresponding PR, I'll add it in the Noto Sans PR so we can track the changes better). Otherwise, there are some interpolation issues remaining, and some of them should be corrected ( |
Noto Serif doesn't include a Devanagari subset. (Noto Sans LGC shouldn't include a Devanagari subset, but workspace needed it in the distant past and now Google Fonts has to contain one for ever more. :-) ) That's why it's a smaller set of changes. lslash looks OK to me (better than the "proposed fix"), but germandbls should be fixed. |
It seems to be working ok. Screen.Recording.2024-09-25.at.19.17.55.movWould you like to modify it still? Other than that this LGTM |
Ok, In that case, you would probably like to review the |
Taken from the upstream repo https://github.com/notofonts/latin-greek-cyrillic at commit notofonts/latin-greek-cyrillic@22328ea.