Closed grimen closed 8 years ago
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!
) and we'll verify. Thanks.
@googlebot I signed it!
CLAs look good, thanks!
@katgironpe OK, done.
@katgironpe Reviewed the correction? Would be really useful if this PR could be merged as soon as possible to be aligned with the other iconfonts.
@grimen Reviewed. Almost correct. Please make the change and maybe someone will merge this.
@katgironpe The input seems to be incorrect though, I use .bin
in all projects - not specifying the dot breaks it because the local folder is named .bin
(hidden) - not bin
.
@grimen you're right. I just checked. Sorry bout that. Looks good.
Can someone merge this? It would really be helpful to have SVG. Primary reason: many of us merge all fonts we use and it easiest with SVG file available. Though it's not much of a performance benefit for some, it is for us.
Hi there,
I'm not going to be merging this because it's easier for us to add SVG font generation on our end of things.
That said, we're going to be pushing a release this week that includes SVG fonts.
@shyndman Great! Thanks. So @grimen did much as we've been looking for ways to convert the fonts to SVG. His solution seems to work. I'd help test it out. Thank you, both.
@shyndman @katgironpe Any update? This issue is currently blocking in the GlyphIcons project.
It's now soon 2 months since this PR was created, soon 1 month since i asked about status referencing you. Still no single word as a reply even if requested.
Hey @grimen,
Sorry for not getting back. I added an SVG font for you a couple weeks ago, but forgot to reply.
https://github.com/google/material-design-icons/blob/master/iconfont/MaterialIcons-Regular.svg
Everything look good? Can I close this off?
@shyndman OK, got it - that's great!
added SVG font format + gulp build task (TTF to SVG transformation). fixes #149