search

brackets-userland / brackets-npm-registry

Extension to install other extensions with npm
MIT License
44 stars 8 forks source link

replacing existing icon with one that better matches the brackets design #39

Closed catdad closed 8 years ago

catdad commented 8 years ago

Fixes #38.

I added a new icons for the default and active states. It uses a Brackets-style "lego block" icon (like the default plugin icon), with NPM's "n" icon inside it. It's too cramped and unreadable to fit 3 letters inside a 20x20 icon, and the "n" icon is commonly used for smaller NPM icons.

Links to the icons: default gray icon active icon

zaggino commented 8 years ago

:+1:

zaggino commented 8 years ago

Published as 0.5.1

zaggino commented 8 years ago

@catdad not sure if it's just me, or is the normal registry icon bigger (height)?

image

catdad commented 8 years ago

It shouldn't be. The red doesn't stand out well against the gray, which makes the icon look slimmer all around. I actually had to make the red a lot brighter than the standard NPM red color. The gray and red icons both use the exact same block shape.

catdad commented 8 years ago

Actually, you are not necessarily wrong. You might be onto something.

sizes

The normal registry icon is not taller, but it is about a pixel down from the one I made. I can fix that.

zaggino commented 8 years ago

Thanks, it just popped into my eyes, I wasn't measuring anything :)