-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tomato, cherry tomato #5
Comments
I probably should change the leaves on the cherry tomatoes, they look too uniform. Also, they look like frolicking Patricks 😜 |
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? |
Could be pretty to have a sight ripening gradient down the vine too? On Jan 19, 2016, 9:31 AM -0800, [email protected], wrote:
|
Sounds good, I'll do that later!
|
Not sure? Is there a benefit to making a rule of that? |
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. |
It had come up here: openfarmcc/OpenFarm#685 (comment), 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. |
To be honest, I don't like those cherry tomatoes with a gradient. The light pink flare looks a lot better, though. |
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. |
Attached is a zip with the two SVGs, tomato with pink flare and cherry tomatoes in one color. |
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: Thanks for the offer! If you want, I can write an imagemagick shell script and submit it via PR...? |
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 should we make sure we add something to the repo before we close an issue? Otherwise it quietly fade away. |
Oh, totally, my bad @simonv3, I had it in my head this was merged and done. Reopened. |
The text was updated successfully, but these errors were encountered: