Add support for srcset

[akesser]

I added support for comma separated lists in attributes to be able to handle srcset
For example

<img src="image.jpg" srcset="image.jpg 1x, [email protected] 2x">

is converted to

"<img src=\"" + require("./image.jpg") + "\" srcset=\"" + require("./image.jpg") + " 1x, " + require("./[email protected]") + " 2x \">"

Gruß

-akesser-

[sokra]

Good idea, but I think this could be solved in a more elegant way.

Try to leave all existing code as it was before and only modify the links array before this line: https://github.com/webpack/html-loader/pull/55/files#diff-168726dbe96b3ce427e7fedce31bb0bcR44

This way we don't need to touch the additional stuff after the url.

And please add some tests.

[akesser]

As you suggested I implemented the srcset support by only modifying the links array. And I added some tests.

[frutality]

Any news on merging? Just googled this issue. Srcset is very useful attribute, and we really can use it according to caniuse. Most of default retina-devices browsers support it.

[xzilja]

Any idea on when this pull request will be merged?

[niieani]

You know - what would be even more awesome is if the loader could automatically generate a srcset for multiple target resolutions, in case the original image is large.

[joshwiens]

@iljadaderko - Shortly
@niieani - Throw up an issue for discussion, that sounds like a handy feature.

@hemanth - LGTM though a second set of eyes would be helpful.

[joshwiens]

@akesser - Please pull in the latest changes from master to enable all the Travis validations prior to merge.

[hemanth]

@d3viant0ne #66 needs a rebase.

[joshwiens]

#66 can be closed. Travis updates in master accomplished the same thing

[hemanth]

Done. @d3viant0ne COVERALLS setup is not up yet.

[hemanth]

@akesser TypeError: Cannot read property 'hash' of undefined failing the test.

[akesser]

I updated my index.js file to pass the nom test