w0rm / gulp-svgstore

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

Use <symbol> instead of <g>? #1

Closed martinwolf closed 10 years ago

martinwolf commented 10 years ago

Hey,

grunt-svgstore switched from <g> to <symbol>. Any chance you will, too? Chris Coyier wrote about why it's better: CSS-Tricks

Have a nice day — Martin

w0rm commented 10 years ago

@martinwolf hi, sorry for the late reply. This is really interesting, especially that viewBox won't be needed. I will try to implement it asap.

martinwolf commented 10 years ago

@w0rm Oh, I totally missed that. Thank you! :+1:

w0rm commented 10 years ago

@martinwolf you're welcome! Feel free to open more issues. Also I updated readme with more usage examples today.

martinwolf commented 10 years ago

@w0rm Nice! I'll get in touch if I run in to errors or problems.