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

Update header.tpl #112

Closed
wants to merge 1 commit into from
Closed

Update header.tpl #112

wants to merge 1 commit into from

Conversation

Madekozu
Copy link

Current browsers prevent unsafe content from being reloaded (http >> https)

@Madekozu Madekozu closed this May 30, 2022
@Madekozu Madekozu deleted the patch-1 branch May 30, 2022 18:31
@azett
Copy link
Member

azett commented May 31, 2022

Hi, thank you very much for the PR! I'd like to go a step further - wouldn't that be a great opportunity to include the font file into the style directly instead of loading it from Google? What do you think?

@Madekozu
Copy link
Author

Yes, this is definitely the better way to integrate the fonts locally.

@azett
Copy link
Member

azett commented Jun 23, 2022

Completely removed unneccessary font resource with 937b566 - thanks for suggesting, @Madekozu!

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