Jonathan Kemp

Results 128 comments of Jonathan Kemp

This option doesn't work? https://github.com/jonkemp/gulp-inline-css#optionsremovelinktags

Oh I see what you are saying. Seems like `data-embed` is the solution here but maybe it doesn't work for the link tag.

Possibly. I would suggest providing a reduced test case with gulp. https://css-tricks.com/reduced-test-cases/

`removeStyleTags` should be set to true. https://github.com/jonkemp/gulp-inline-css#optionspreservemediaqueries

Could you provide more information? Also, could you update your example because it shows that you have `removeStyleTags` set to false.

One thing is you are missing some node modules in the package.json, ``` gulp-notify bs-snippet-injector ``` but I am seeing the same problem. I'll look into it.

No, I don't really understand what is wrong. The tests pass so I know it works to some extent. Sorry.

The version of this plugin is 3.0 on npm. https://www.npmjs.com/package/gulp-inline-css Maybe you are getting the gulp plugin confused with the actual node module is uses? https://www.npmjs.com/package/inline-css

You are correct. The README was changed after it was published to npm

The breaking change is that the inline-css module was updated to use Promises. I bumped the version because I was afraid that may cause some issues.