-
Notifications
You must be signed in to change notification settings - Fork 71
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
move fonts.googleapis.com to local cdn #88
Comments
PR welcome to include the font in the theme 👍 |
I tried to follow this but I seem to have just inlined them... |
I think this should be the solution - the font is delivered from the website where grav is installed. The performance will be better and the dependency of a second (font) server which must be available is gone. |
@flaviocopes What do you think of their solution? Possibly on the agenda or nah? |
Is there any update on this issue? |
Hi there, |
I use grav on localhost and have often issues during train rides. The access to fonts.googleapis.com does often not work or delays the delivery of the pages.
In my opinion, the fonts should be embedded to improve performance and anonymization.
A good documentation can be found here: https://www.keycdn.com/blog/google-fonts-cdn/
Maybe it's best to embedded the functions in core, so the improvement is available for all themes. But the problem exists actually in the standard theme.
What's your opinion?
The text was updated successfully, but these errors were encountered: