Closed (fixed)
Project:
Professional Theme
Version:
7.x-2.02
Component:
Code
Priority:
Minor
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
27 Mar 2013 at 01:45 UTC
Updated:
10 Nov 2015 at 20:56 UTC
Jump to comment: Most recent
@import url("http://fonts.googleapis.com/css?family=Droid+Sans:regular,bold|Droid+Serif:regular,italic,bold,bolditalic&subset=latin");
needs to change to
@import url("https://fonts.googleapis.com/css?family=Droid+Sans:regular,bold|Droid+Serif:regular,italic,bold,bolditalic&subset=latin");
I believe you can even remove the https:// part but didnt try it.. figure might as well load it in https always
Comments
Comment #1
mheinke commentedthank you for the fix,
i am adding this to the next release
Comment #2
mheinke commentedupdated in version 1.7 :)
thank you!
Comment #3
mheinke commentedComment #4
KeepXtreme commented@import url("https://fonts.googleapis.com/css?family=Droid+Sans:regular,bold|Droid+Serif:regular,italic,bold,bolditalic&subset=latin");should be:
@import url("//fonts.googleapis.com/css?family=Droid+Sans:regular,bold|Droid+Serif:regular,italic,bold,bolditalic&subset=latin");cause: in this format, the connection (http/https) depends on the connection to the server holding the website. Thus a http-site is not slowed down by the https handshake to the google servers which means an improvment in the site load speed as this is passed primary accordingly to tools.pingdom.com
Reference: http://stackoverflow.com/questions/4659345/is-there-any-downside-for-usi...
also use
instead of @import:
http://www.stevesouders.com/blog/2009/04/09/dont-use-import/
or even better, use the drupal function to load external css:
drupal_add_css('//fonts.googleapis.com/css?family=Droid+Sans:regular,bold|Droid+Serif:regular,italic,bold,bolditalic&subset=latin', array('type' => 'external'));
referer: https://drupal.org/node/171209
Comment #5
KeepXtreme commentedComment #7
mheinke commentedComment #8
mheinke commented