w0rm / gulp-svgstore

Combine svg files into one with symbol elements
https://www.npmjs.com/package/gulp-svgstore
646 stars 33 forks source link

Adding example from Issue to docs #18

Closed lukasoppermann closed 9 years ago

lukasoppermann commented 9 years ago

This example makes it much more clear how to use transformSvg when not using the inject method.

lukasoppermann commented 9 years ago

Okay, well, I just used the function from your docs. My point is basically, that it should be noted, that an transformsvg function can be added this way. What would you recommend as a change so you would merge it?

w0rm commented 9 years ago

@lukasoppermann my example from docs passes svg to the callback:

function transformSvg ($svg, done) {
  $svg.attr('style', 'display:none')
  done(null, $svg)
}

Can you update your example with this transformSvg?

lukasoppermann commented 9 years ago

Sure, just like this?