Skip to content

Moved to CDN links for cmunss.otf #178

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

Merged
merged 1 commit into from
Feb 6, 2014

Conversation

rgbkrk
Copy link
Contributor

@rgbkrk rgbkrk commented Feb 6, 2014

This swaps the font out to use a version that is one hosted on a CDN, as suggested in jupyter/nbviewer#185.

We could put a CNAME on the container if you want this to be a shorter URL.

@rgbkrk
Copy link
Contributor Author

rgbkrk commented Feb 6, 2014

This takes the actual fetching of the font down from around 350ms to less than 100 ms, which may get even better as the file propagates across the edge nodes.

screen shot 2014-02-06 at 1 50 54 pm

Here's a Live Demo.

The order in which this is requested by the browser matters here too (still way late into the overall network flow), which means other page optimizations are in order.

@CamDavidsonPilon
Copy link
Owner

❤️ Thanks @rgbkrk!

CamDavidsonPilon added a commit that referenced this pull request Feb 6, 2014
Moved to CDN links for cmunss.otf
@CamDavidsonPilon CamDavidsonPilon merged commit 854ba3e into CamDavidsonPilon:master Feb 6, 2014
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