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

Consider removing Typekit Fonts #3939

Closed
davidfischer opened this issue Apr 11, 2018 · 0 comments
Closed

Consider removing Typekit Fonts #3939

davidfischer opened this issue Apr 11, 2018 · 0 comments
Assignees
Labels
Design Design or UX/UI related Needed: design decision A core team decision is required

Comments

@davidfischer
Copy link
Contributor

Currently, readthedocs.org uses Adobe Typekit to load some fonts. These fonts are Inconsolata, a SIL open font licensed monospace font and four versions of FF Meta, a proprietary font that is also used on the Read the Docs logo.

I know there is some redesign work going on but I think we can get a quick win by swapping these fonts to using a native font stack for a couple reasons.

  • Read the Docs won't look different in development (where TypeKit isn't available) vs production
  • There are performance improvements from not having to download fonts although they aren't huge on readthedocs.org (they're much larger on the RTD theme). It's ~115kb and can be around 1/3 of a second. If you see the "flash of unstyled text" when loading readthedocs.org, this is why.
  • Some users consider Typekit a "tracker" and are concerned about its security and privacy implications. Regardless of your thoughts on that subject, it is 3rd party code we are currently executing on Read the Docs. If we don't use these fonts, we don't need TypeKit.

The biggest downside is that the Read the Docs logo will use a different font than the rest of the site.

I'd be happy to put together a small PR swapping to a native font stack. I believe it will only touch 1 file and be only a few lines. Thoughts?

I detailed some of my thoughts on native font stacks in a ticket on the RTD theme (readthedocs/sphinx_rtd_theme#524).

@davidfischer davidfischer added Design Design or UX/UI related Needed: design decision A core team decision is required labels Apr 11, 2018
@davidfischer davidfischer self-assigned this Apr 11, 2018
davidfischer added a commit to davidfischer/readthedocs.org that referenced this issue Apr 20, 2018
- Replace fonts with a native font stack
- Fixes readthedocs#3939
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Design Design or UX/UI related Needed: design decision A core team decision is required
Projects
None yet
Development

No branches or pull requests

1 participant