Skip to content
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

Source Arabic font from CDN, newer repo #5053

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mapmeld
Copy link
Contributor

@mapmeld mapmeld commented Jan 17, 2025

As discussed in #4893, Google stopped updating our go-to fonts repo https://github.com/notofonts/noto-fonts two years ago. They now distribute multilingual Noto through individual repos, the jsdelivr CDN, and GitHub Pages.

In #5052 we added a Python script to download fonts. For now I suggest a separate list of fonts to download from the new sources (NEWER_REPO). As written this would work for most fonts, though there are exceptions (such as the UI fonts and NotoSansKayahLi) which were handled in #4893.

I'd like to update Arabic first here to fix a known issue in Kurdish map labels. I checked with a maintainer in the Noto Arabic repo, and "Arabic UI" is no longer updated; we also want to switch over to the standard font.

@imagico
Copy link
Collaborator

imagico commented Jan 19, 2025

I am wondering if maintenance wise it is good to implement the path name logic (which is non-standardized and can change at any moment because someone feels the urge to scratch an itch) in script or if it is better to maintain a list of literal URLs or format strings.

This is ultimately something to decide when implementing point 3 on #5043.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants