search

Automattic / gridicons

The WordPress.com icon set
http://automattic.github.io/gridicons/
GNU General Public License v2.0
111 stars 13 forks source link

New Icon: WooCommerce #292

Closed jameskoster closed 6 years ago

jameskoster commented 6 years ago

We're using Gridicons in the wc-admin project we're currently working on and have a need for a Woo bubble icon. Is that the sort of thing that makes sense to add to Gridicons? If so I'll open a PR, if not we'll work something else out :)

@folletto @drw158

folletto commented 6 years ago

If it's a logo, it's usually better suited for https://github.com/Automattic/social-logos :)

Note that the "W" is instead in Gridicons because it has been tweaked (weight, size, balance) to be aligned with the Gridicons standard, so it's not anymore exactlu "the logo", and that's why it's also named "my sites".

jameskoster commented 6 years ago

Yeah I thought as much, but it's a little grey as the Jetpack logo is in there as well.

I'll wait for more consensus, the icon is ready to go either way.

folletto commented 6 years ago

Yeah I thought as much, but it's a little grey as the Jetpack logo is in there as well.

Similar to "My Sites", that's not Jetpack but "Plans" :)

If there's a specific meaning then the Woo bubble icon can be in. It has to be first a meaning, and then "Woo" is one of the possible symbols for that meaning.

If the meaning applies, then the icon is added tweaked to be aligned with Gridicons standards.

jameskoster commented 6 years ago

Yeah I don't think there's any meaning.

Let's add to Social Icons.