Minimize SVG #4

Merged
merged 3 commits into from Mar 16, 2012

Conversation

Projects
None yet
4 participants
Contributor

sindresorhus commented Mar 14, 2012

Redid the SVG by hand and made it correct, pixel perfect and small.

@sindresorhus sindresorhus Minimize SVG
Redid the SVG by hand and made it correct, pixel perfect and small.
06c66b4

lgarron commented Mar 14, 2012

You're braver than I am. :-P

Fortunately, the rounded rect construct works well in SVG, but your solid version has white layering instead of transparency for the M and the arrow. Is there a "nice" way to do something like this?

Also, you changed the dimensions to 210*130. While I also think it might be good to think about tweaking the grid or the dimensions, the original version had an exact 1.675:1 ratio, which scales very well with powers of 2. I would be strongly in favor of keeping that.

Contributor

sindresorhus commented Mar 14, 2012

Used a mask to fix the transparency and reverted back to 208x128 ;)

lgarron commented Mar 14, 2012

Cool, I'm all for replacing the current versions with these.

I didn't even know about using masks in SVG. However, this means the shape only shows up as a solid rounded rectangle in Inkscape. Any way to make it semantically neat but also easy to edit (e.g. to change the shade)?

Contributor

sindresorhus commented Mar 14, 2012

I originally only tested with browsers and Illustrator CS5. But downloaded Inkscape 0.48 on my Mac right now and opened the file, and it looks exactly like in the browser, and I'm able to change the color. Are you sure you're running the latest version?

lgarron commented Mar 14, 2012

I'm on Inkscape 0.48.2 and I just get this:

Markdown-solid.svg in Inkscape

Contributor

sindresorhus commented Mar 14, 2012

I can't explain that...

My Inkscape:
screenshot

Owner

dcurtis commented Mar 15, 2012

This is really great. What's the verdict? I'll merge it in if it's better (it seems to be much more compact).

Smaller + Equally precise = better.

With the exception of missing description metadata, if you wanted that.

dcurtis merged commit 3525fd2 into dcurtis:master Mar 16, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment