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

Google font RGPD legal issue #18

Closed
LtSich opened this issue Apr 4, 2023 · 3 comments
Closed

Google font RGPD legal issue #18

LtSich opened this issue Apr 4, 2023 · 3 comments
Assignees

Comments

@LtSich
Copy link

LtSich commented Apr 4, 2023

Hi,

I will just link the same issue opened on the original project here : poole/hyde#298.

The idea is to replace that:

<link rel="stylesheet" href="https://fonts.googleapis.com/css?family=PT+Sans:400,400italic,700|Abril+Fatface">

With a local font that don't need to connect to Google server.

@ArnaudLigny ArnaudLigny self-assigned this Apr 4, 2023
@ArnaudLigny
Copy link
Member

ArnaudLigny commented Apr 6, 2023

Hello @LtSich, this is an interesting topic.

I had decided to keep using Google Fonts for performance reasons, but it's true that not depending on a third-party service seems a relevant option.

Also, I added an option (release 2.2.0) to modify this setting in order to use local fonts:

hyde:
  fonts: local # google or local

@ArnaudLigny
Copy link
Member

See the following partial template for details: https://github.com/Cecilapp/theme-hyde/blob/master/layouts/partials/fonts.html.twig

@LtSich
Copy link
Author

LtSich commented Apr 6, 2023

Nice thx !
I will take a look at that soon.

@LtSich LtSich closed this as completed Apr 6, 2023
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

2 participants