-
Notifications
You must be signed in to change notification settings - Fork 646
Add js license info for example page. #311
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
Conversation
</div> | ||
<script type="text/javascript" src="http://code.jquery.com/jquery-2.1.1.min.js"></script> | ||
<script type="text/javascript" src="http://cdn.jsdelivr.net/g/jquery.ui@1.10"></script> | ||
<script type="text/javascript" src="https://code.jquery.com/jquery-2.1.1.min.js"></script> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason you changed these to https? https can cause up to 5x slower load, and i personally don't see a reason that one would need to load these over https, its jquery.
I can see a case for using :// instead as that would remove it being blocked when going to browsing the https:// site.
Personally, I always try to use HTTPS when possible. For me it's like turning the lights off when leaving a room. I wasn't aware of the speed difference though and I suppose there is no harm in using HTTP for the jquery URLs. "I can see a case for using :// instead as that would remove it being blocked when going to browsing the https:// site." OK, I'll change all the URLs to that. |
Sorry it's actually // not :// On Sun, Mar 22, 2015 at 12:03 AM, albsod notifications@github.com wrote:
|
Add js license info for example page.
This is a follow-up to #257.