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

Change jQuery CDN to a China-safe option #1739

Closed
wants to merge 1 commit into from

Conversation

roblarsen
Copy link
Member

Fixes: #1737

China, with with more web users than there are people in the US, blocks googleaplis.com. We really ought to put the "WORLD" back in World Wide Web with a China-friendly option for jQuery.

@hatzopoulos
Copy link

FWIW I once tested a few of them and jQuery's (maxcdn) was faster at the time. Although my testing method was rather crude and done 3 years ago. http://stackoverflow.com/a/12981092/881551

@chrislaughlin
Copy link

👍 looks good and I assume that the official jQuery source would be the best.

@roblarsen
Copy link
Member Author

I forgot about this whole discussion. @alrra? (or whoever else has the keys these days...)

@alrra
Copy link
Member

alrra commented Jan 5, 2016

@roblarsen Can you also update:

  • the docs (see: html.md & faq.md)
  • the commit message to include a little bit more information about why this change is made and that it doesn't have any downsides compared to the current solution (see also the previous discussion: Drop Google's CDN for jquery.com? #1191).

Thanks!

@roblarsen roblarsen force-pushed the use-jquery-cdn branch 2 times, most recently from 6470fbd to c5638cc Compare January 5, 2016 17:20
This commit closes h5bp#1737

In countries where googleapis.com is blocked (China and other countries)
H5BP is broken out of the box as jQuery will not load until the request
times out. This commit changes the default jQuery source to use the
jQuery CDN. The jQuery CDN is as fast or faster (by some reckoning) than
the Google Hosted Libraries version and is available worldwide. While
the chance of hitting the cache lottery is smaller with the jQuery CDN
(it's not as ubiquitous as Google) that chance is not enough to override
the need to ship working code for everyone in the world.
@roblarsen
Copy link
Member Author

👍 Updated

@alrra alrra closed this in 340b012 Jan 9, 2016
@alrra
Copy link
Member

alrra commented Jan 9, 2016

@roblarsen Thanks! 💜

Merged with some modifications (e.g.: also updated the CHANGELOG.md) in 340b012.

@roblarsen
Copy link
Member Author

👍

eleanor-byhook pushed a commit to eleanor-byhook/html5-boilerplate that referenced this pull request Feb 29, 2016
In countries where `googleapis.com` is blocked (e.g.: China),
HTML5 Boilerplate is broken out of the box as jQuery will not load
until the request times out.

This commit changes the default jQuery source to use the jQuery CDN
as it is as fast or faster (by some reckoning) than the Google Hosted
Libraries version and is available worldwide.

While the chance of hitting the cache lottery is smaller with the
jQuery CDN (it's not as ubiquitous as Google), that chance is not enough
to override the need to ship working code for everyone in the world.

- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -

Fix h5bp/html5-boilerplate#1737
Close h5bp/html5-boilerplate#1739
@roblarsen roblarsen deleted the use-jquery-cdn branch August 23, 2016 19:20
jeffreznik pushed a commit to jeffreznik/sw-test that referenced this pull request Oct 15, 2017
In countries where `googleapis.com` is blocked (e.g.: China),
HTML5 Boilerplate is broken out of the box as jQuery will not load
until the request times out.

This commit changes the default jQuery source to use the jQuery CDN
as it is as fast or faster (by some reckoning) than the Google Hosted
Libraries version and is available worldwide.

While the chance of hitting the cache lottery is smaller with the
jQuery CDN (it's not as ubiquitous as Google), that chance is not enough
to override the need to ship working code for everyone in the world.

- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -

Fix h5bp#1737
Close h5bp#1739
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.

4 participants