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

Self-host fonts, add option to not copy to directory #1189

Closed
wants to merge 1 commit into from
Closed

Self-host fonts, add option to not copy to directory #1189

wants to merge 1 commit into from

Conversation

okaneco
Copy link
Contributor

@okaneco okaneco commented Apr 14, 2020

Vendors Open Sans and Source Code Pro as woff/woff2, adds respective licenses for each.
Supports character sets for:
- open-sans-v17-latin_vietnamese_latin-ext_greek-ext_greek_cyrillic-ext_cyrillic
- source-code-pro-v11-latin_vietnamese_latin-ext_greek_cyrillic-ext_cyrillic
Add option to disable inclusion of woff/woff2

Open Sans is licensed under Apache 2.0, Source Code Pro is licensed under SIL Open Font License. The fonts were downloaded with google-webfonts-helper.

I added controls to the HTML config struct to make the fonts optional. By default, the fonts are enabled with both woff2/woff files. Users can disable the self-hosted fonts, or disable only the woff format files which are supported by older browsers and larger in size.

[output.html.font]
enable = true
woff = true

Vendors Open Sans and Source Code Pro as woff/woff2, adds respective licenses for each.
Supports character sets for:
    - `open-sans-v17-latin_vietnamese_latin-ext_greek-ext_greek_cyrillic-ext_cyrillic`
    - `source-code-pro-v11-latin_vietnamese_latin-ext_greek_cyrillic-ext_cyrillic`
Add option to disable inclusion of woff/woff2

Open Sans is licensed under Apache 2.0, Source Code Pro is licensed under SIL Open Font License.
@ehuss
Copy link
Contributor

ehuss commented May 20, 2020

Fixed via #1188.

@ehuss ehuss closed this May 20, 2020
@okaneco okaneco deleted the remove branch May 20, 2020 18:33
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