Combine Google Fonts inside 1 Request?

Posted in: Newspaper
Post count: 13

OK so I am trying to optimize my website and after checking Pingdom I see this (right now just focusing on the Google part outlined in red)

googleerror

I am wanting to combine the 2 Google fonts in to 1 request, after checking Googles own documentation I find this solution:

fix

According to Google I can take the 2 requests

http://fonts.googleapis.com/css?family=Open+Sans%3A300italic%2C400italic%2C600italic%2C400%2C600%2C700&ver=4.7
http://fonts.googleapis.com/css?family=Roboto%3A500%2C400italic%2C700%2C900%2C500italic%2C400%2C300&ver=4.7

and do something like this I think (syntax might be wrong):

<link rel="stylesheet" type="text/css" href="http://fonts.googleapis.com/css?family=Open Sans|Roboto">

to produce just 1 request, but my question is where do I put that new code to make newspaper only issue the 1 request? Please help me, thank you in advance.

Post count: 7909

Hi,

Please note that we don’t provide any type of custom services, so if you want to modify this and you have another solution here you can find the default fonts – http://screencast.com/t/xjCjYaEIhttp://screencast.com/t/3zBmB0Wc

Thanks!

Post count: 13

It isn’t a custom service, your theme is spitting out errors on site tests and I am just asking you all for support how to correct it since support comes with the purchased theme.

I looked at what you provided the location of the code, thank you, based on what the Google documentation this is what I done:

combined

after saving and replacing the old td_config the site quits working and only displays a white screen. Am I on the right track here? If we can combine these in to 1 link then that will save a round trip to the server, making the theme a little bit faster, and 1 less error in the code, we could then make this correction part of the default newspaper theme I would imagine.

Post count: 53

Yes this feature must be implemented

Post count: 7909

Hi,

I don’t know how you implement this but it works on my side – http://screencast.com/t/bDOQTHFNwhttp://screencast.com/t/vraAnUahPo
I will add it on our list and it will be considered for future. From what I remember in old theme version we had this kind of implementation but I cannot say for sure why it was removed. The developers will check this suggestion and if it doesn’t create any issues it will be implemented.

Thanks!

Post count: 13

Alln great!

Can you screenshot your td_config? The Google Fonts portion. After you made the changes. In your last screen shot I do not see a separator “|” but according to Google documentation you need to use it.

Post count: 7909

Hi,

I have already provided a screenshot with td_config file above – http://screencast.com/t/bDOQTHFNw

Thanks!

Viewing 7 posts - 1 through 7 (of 7 total)
You must be logged in to reply to this topic.