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

Serve Source Sans locally - avoid law suits #2077

Open
koppor opened this issue Dec 4, 2022 · 3 comments
Open

Serve Source Sans locally - avoid law suits #2077

koppor opened this issue Dec 4, 2022 · 3 comments

Comments

@koppor
Copy link

koppor commented Dec 4, 2022

Is your feature request related to a problem? Please describe.

In Germany, it is forbidden to fetch Fonts from Google. If one doesn't follow this rule, one has to play (at least) 300€ per case. See https://ll-ip.com/aktuelles/raag-kanzlei-dikigoros-nikolaos-kairis/ for details.

Describe the solution you'd like

Storefront should ship the font in its distribution.

Describe alternatives you've considered

There is a hint at https://wordpress.org/support/topic/css-font-display-how-to-add/ to create a child theme. It seems, however that 300+ lines of code need to be copied - only for replacing one font.

@dinhtungdu
Copy link
Member

dinhtungdu commented Dec 6, 2022

Hi @koppor,

Thanks for your report. I acknowledge the issue of Google Fonts and GDPR and I can confirm Storefront is loading Source Sans directly from Google Fonts as you described.

Storefront should ship the font in its distribution.

Thanks for your suggestion. Our team will need to discuss this before taking any action. Because Storefront is now in maintenance mode, the process can take a while.

In the meantime, you can use a plugin to make your site GDPR-compliant: https://wordpress.org/plugins/local-google-fonts/. Can you please test that plugin to see if it solves your issue?

@dinhtungdu dinhtungdu added the needs feedback The issue/PR needs a response from any of the parties involved in the issue. label Dec 6, 2022
@koppor
Copy link
Author

koppor commented Dec 8, 2022

Thank you for the quick response. The plugin works great, thank you for the hint!

@dinhtungdu dinhtungdu removed the needs feedback The issue/PR needs a response from any of the parties involved in the issue. label Dec 8, 2022
@Aljullu
Copy link
Contributor

Aljullu commented Dec 9, 2022

Heads-up that in #845 (comment) there is a suggested snippet that might also work. There is also this filter that might be useful.

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

No branches or pull requests

3 participants