Pagespeed performance, inline js and css

giacomosilli · October 24, 2018, 8:30am

I’m trying to maximize the performance of my blog.

Pagespeed complains a bit about everything, I’m trying to make it happy.

Adding the content of the file ghost-sdk.min.js and built / screen.css directly into the default.hbs file is a bad practice or does not involve particular annoyances? thank you

Kevin · October 24, 2018, 10:41am

It’s not possible to disable the injection of the ghost-sdk.min.js file at the moment due to backwards-compatibility although we hope to be able to remove it as a breaking change in Ghost 3.0.

We are looking at ways to improve it’s effects on performance audits in the meantime.

github.com/TryGhost/Ghost

Optimise performance of ghost-sdk.js in themes

opened 04:36AM - 12 Oct 18 UTC

closed 05:18PM - 02 Jan 19 UTC

JohnONolan

server / core

Currently we inject ghost-sdk.js into themes as follows: ``` <script src="/p…ublic/ghost-sdk.min.js?v=30b82fce68"></script> <script> ghost.init({ clientId: "ghost-frontend", clientSecret: "f84a07a72b17" }); </script> ``` and Google Pagespeed doesn't like it ![image](https://user-images.githubusercontent.com/120485/46848106-3775cb80-ce12-11e8-84ae-a4e0850c4641.png) --- ## Question 1: Can we add `async` to the script? or, preferably: ## Question 2: Could we not just inline the whole thing? It's tiny anyway, and would actually be a reduction of 2 script tags to 1: ``` <script> !function(){var o,i,n,e,r="https://demo.ghost.io/ghost/api/v0.1/";n={api:function(){var n,e=Array.prototype.slice.call(arguments),t=r;return(n=e.pop())&&"object"!=typeof n&&(e.push(n),n={}),(n=n||{}).client_id=o,n.client_secret=i,e.length&&e.forEach(function(n){t+=n.replace(/^\/|\/$/g,"")+"/"}),t+function(n){var e,t=[];if(!n)return"";for(e in n)n.hasOwnProperty(e)&&(n[e]||!1===n[e])&&t.push(e+"="+encodeURIComponent(n[e]));return t.length?"?"+t.join("&"):""}(n)}},e=function(n){o=n.clientId?n.clientId:"",i=n.clientSecret?n.clientSecret:"",r=n.url?n.url:r.match(/{\{api-url}}/)?"":r},"undefined"!=typeof window&&(window.ghost=window.ghost||{},window.ghost.url=n,window.ghost.init=e),"undefined"!=typeof module&&(module.exports={url:n,init:e})}(); ghost.init({ clientId: "ghost-frontend", clientSecret: "f84a07a72b17" }); </script> ``` ## Question 3: Could this also move to `</body>` instead of `</head>` ?

giacomosilli · October 24, 2018, 11:18am

Thanks for the reply,
I read this post and I tried to enter the code directly, I have not found any problem.

I had thought to leave it so momentarily if I did not risk compromising the system. Any problems?

And what about the css, is it bad practice?

milessmith · October 24, 2018, 5:08pm

yes man you are right i also want to know Am facing the same problem. Hope anyone here can help me too. Thanks in advance.
Regards,
miles smith
ViaMichelin PutLocker Google Earth

giacomosilli · October 25, 2018, 5:15pm

?

system · November 8, 2018, 5:26pm

This topic was automatically closed 14 days after the last reply. New replies are no longer allowed.

Topic		Replies	Views
Page Speed with scripts Developer help	1	633	January 26, 2019
Optimising a ghost blog for speed Developer help	5	4103	July 2, 2020
Optimizing unused JS? Using Ghost	4	253	September 10, 2023
How to fix bad CSS in Ghost? Developer help	1	469	May 19, 2020
Ghost does not serve Tailwind CSS files Developer help	1	918	February 20, 2022

Pagespeed performance, inline js and css

Related Topics