open-crop-icons icon indicating copy to clipboard operation
open-crop-icons copied to clipboard

Tomato, cherry tomato

Open jneubrand opened this issue 9 years ago • 19 comments

cherrytomatoes tomato

jneubrand avatar Jan 19 '16 05:01 jneubrand

I probably should change the leaves on the cherry tomatoes, they look too uniform. Also, they look like frolicking Patricks :stuck_out_tongue_winking_eye:

jneubrand avatar Jan 19 '16 05:01 jneubrand

c39f0d9c-be2a-11e5-9ccf-04c0a6068c00 c3a09eaa-be2a-11e5-9d54-df1fe9b2834d

Great stuff! I'm throwing in these resized to 24px just so people can see them small.

Maybe bring the highlight on the tomato down to a light pink so it looks a bit less glossy? Also the natural stagger you've given the cherry toms gets lost at smaller sizes. Maybe exaggerate it a bit?

andru avatar Jan 19 '16 10:01 andru

These look great. Thanks!

If the tomato white gets set to light pink that makes it 4 colors - are we trying to limit to 3?

simonv3 avatar Jan 19 '16 17:01 simonv3

Could be pretty to have a sight ripening gradient down the vine too?

On Jan 19, 2016, 9:31 AM -0800, [email protected], wrote:

These look great. Thanks!

If the tomato white gets set to light pink that makes it 4 colors - are we trying to limit to 3?

— Reply to this email directly orview it on GitHub(https://github.com/openfarmcc/open-crop-icons/issues/5#issuecomment-172927086).

warpling avatar Jan 19 '16 17:01 warpling

http://farm5.static.flickr.com/4115/4825175507_3ea3cf73dd_m.jpg

warpling avatar Jan 19 '16 17:01 warpling

Sounds good, I'll do that later!

On Jan 19, 2016, at 09:42, Ryan McLeod [email protected] wrote:

Could be pretty to have a sight ripening gradient down the vine too?

On Jan 19, 2016, 9:31 AM -0800, [email protected], wrote:

These look great. Thanks!

If the tomato white gets set to light pink that makes it 4 colors - are we trying to limit to 3?

— Reply to this email directly orview it on GitHub(https://github.com/openfarmcc/open-crop-icons/issues/5#issuecomment-172927086).

— Reply to this email directly or view it on GitHub.

jneubrand avatar Jan 19 '16 17:01 jneubrand

If the tomato white gets set to light pink that makes it 4 colors - are we trying to limit to 3?

Not sure? Is there a benefit to making a rule of that?

andru avatar Jan 19 '16 22:01 andru

I don't think a hard rule is necessary, but I think less is more. Over time we could create an icon style guide to help get some consistency.

roryaronson avatar Jan 19 '16 23:01 roryaronson

It had come up here: https://github.com/openfarmcc/OpenFarm/issues/685#issuecomment-172450117, I think it makes them easier to scale to smaller sizes. Maybe not a hard rule, but keeping the amount of colours down. I don't know if most plants will need more than 3 or 4 to be recognizable.

simonv3 avatar Jan 20 '16 00:01 simonv3

tomato cherrytomatoes

jneubrand avatar Jan 20 '16 02:01 jneubrand

To be honest, I don't like those cherry tomatoes with a gradient. The light pink flare looks a lot better, though.

jneubrand avatar Jan 20 '16 02:01 jneubrand

Here's the 24px versions: tomato 24 cherrytomatoes 24

jneubrand avatar Jan 20 '16 02:01 jneubrand

Sweet! I'm agreed about the gradient on the cherry tomatoes. It's a great idea but I don't think it works too well in practice.

andru avatar Jan 21 '16 12:01 andru

Attached is a zip with the two SVGs, tomato with pink flare and cherry tomatoes in one color. plainsvg-tomato-cherrytomato.zip I release it under CC0 1.0.

jneubrand avatar Jan 21 '16 17:01 jneubrand

Sweet. We're still woking on the processes and whatnot of exactly how we'll export, package, etc icons so I'll leave this open for now.

We'll probably need each icon to be resized to 3 sizes (24,32,64) and submitted with a pull request. I can handle that when the time comes if you like, or if you want to handle it yourself let me know and I'll ping this issue with more info when we've solidified the processes a bit.

andru avatar Jan 21 '16 20:01 andru

@andru: Thanks for the offer! If you want, I can write an imagemagick shell script and submit it via PR...?

jneubrand avatar Jan 24 '16 03:01 jneubrand

I reckon most icons will need to be custom designed for each resolution, so a resize script won't do it in those cases.

andru avatar Jan 24 '16 12:01 andru

@andru should we make sure we add something to the repo before we close an issue? Otherwise it quietly fade away.

simonv3 avatar Jan 31 '16 17:01 simonv3

Oh, totally, my bad @simonv3, I had it in my head this was merged and done. Reopened.

andru avatar Jan 31 '16 17:01 andru