The gulp-resource-hints from theetrain

Change 'pageToken' option

Change pageToken option to just token

Add CI tests

default test
user-defined tests (one per option)

Check for <head> before scanning assets

It may be more efficient to check if the document has a <head> (via soup) before building resource hints, that way I can avoid the 'no head' warning.

Add README

Include

Support all resource hints

Cleanup

simplify merge function to

function merge(opts) {
  var options = Object.assign({}, defaults, opts);
  options.paths = Object.assign({}, defaults.paths, opts.paths);
  return options
}

use new Set() instead of map when parsing assets to automatically remove duplicates from the array
use reduce to append resource hints to data variable

Write out resource hints in a sensible order

In the html file's <head>, resource hints ought to be written out in this order for maximum performance:

preload
prerender
dns-prefetch
preconnect
prefetch

User-submitted entries can accept an object by default, and they'll be written out as above. If they prefer, they can submit an array.

Support basic filetypes

Add 'silent' mode

Add 'silent' option to disable console output.

When custom tokens are detected, use them

Need a hotfix: must return true when insertion point is a token at https://github.com/theetrain/gulp-resource-hints/blob/master/lib/helpers.js#L121

Scan CSS files for static assets

Use list-assets to scan CSS files within captured html streams, and write resource hints for them.

Use commented-out token

Instead of ##gulp-resource-hints## use  for added safety.

Add comments to functions

explain internal functions
briefly explain why I'm using external functions

theetrain / gulp-resource-hints Goto Github PK

gulp-resource-hints's People

Contributors

Stargazers

Watchers

gulp-resource-hints's Issues

Recommend Projects

Recommend Topics

Recommend Org