openfarmcc / open-crop-icons

An openly licensed icon set of commonly cultivated crops.
33 stars 10 forks source link

Tomato, cherry tomato #5

Open jneubrand opened 8 years ago

jneubrand commented 8 years ago

cherrytomatoes tomato

jneubrand commented 8 years ago

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:

andru commented 8 years ago

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?

simonv3 commented 8 years ago

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?

warpling commented 8 years ago

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

On Jan 19, 2016, 9:31 AM -0800, Simonnotifications@github.com, 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 commented 8 years ago

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

jneubrand commented 8 years ago

Sounds good, I'll do that later!

On Jan 19, 2016, at 09:42, Ryan McLeod notifications@github.com wrote:

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

On Jan 19, 2016, 9:31 AM -0800, Simonnotifications@github.com, 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.

andru commented 8 years ago

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?

roryaronson commented 8 years ago

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.

simonv3 commented 8 years ago

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.

jneubrand commented 8 years ago

tomato cherrytomatoes

jneubrand commented 8 years ago

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

jneubrand commented 8 years ago

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

andru commented 8 years ago

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.

jneubrand commented 8 years ago

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.

andru commented 8 years ago

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.

jneubrand commented 8 years ago

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

andru commented 8 years ago

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

simonv3 commented 8 years ago

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

andru commented 8 years ago

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