critical-path-css-demo

A lot going on in this PR, so an explanation is required: Issues: The original documentation example no longer functions as expected because Gulp has evolved. The original code example is outdated. Fixes: Code example and documentation updated to current Gulp best-practice. Code example changed to simplified Bootstrap. Documentation simplified to focus on generic inclusion in existing Gulp project rather than tutorial. Feedback wanted.

Gulp example fails because it uses dest (should be target) and needs an update to meet new Gulp 4 standards. PR incoming.

So, i started looking at the two demos (normal page vs page using critical) and obviously the first i did was open the devtools to see the differences... but i noticed a few odd things : P On https://addyosmani.com/critical-path-css-demo/output/normal/ There is a site.css file being requested (returns 404) using loadCSS. There is actually another file named main.css that loads fine and has the styles for the page. On https://addyosmani.com/critical-path-css-demo/output/critical/ Same as before, an orphan site.css. The main.css file doesn't seem to be loaded via loadCSS (which is what the README mentions) but through a simpler js snippet that creates a link element and injects it. So, i just ran the builds again, for the normal page and for the page with the inlined styles and copied the HTML files into the output folder which holds the demo pages. The new HTML files score in pagespeed insights as expected and throw no errors: Normal https://developers.google.com/speed/pagespeed/insights/?url=https%3A%2F%2Fimagentleman.github.io%2Fcritical-path-css-demo%2Foutput%2Fnormal%2F Critical https://developers.google.com/speed/pagespeed/insights/?url=https%3A%2F%2Fimagentleman.github.io%2Fcritical-path-css-demo%2Foutput%2Fcritical%2F And the critical example page has inlined styles, loadCSS asynchronously loads main.css and the noscript fallback is present, and the normal example page doesn't have any of the optimizations (no async css via loadCSS, etc). This PR doesn't add anything major but might help avoid some confusion on newcomers (like me : S) to the library that inspect the HTML of the demos to see what it does and how it works.

npm install critical --save-dev fails on this file: node install.js Error log attached. 2017-11-14T16_46_06_456Z-debug.log