Skip to content
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

Add tree icon #644

Closed
wants to merge 1 commit into from
Closed

Add tree icon #644

wants to merge 1 commit into from

Conversation

lscheibel
Copy link
Member

I needed this one for a soundboard.

image

@karsa-mistmere
Copy link
Member

All in all nice, but I think the trunk needs to be taller, as deciduous trees' trunks usually are, otherwise it looks a little more like a shrub than it does a tree:
image
image

@lscheibel
Copy link
Member Author

I like the last one, I thought it looked a bit like cotton candy but I think in context it works. I updated the PR

@ericfennis
Copy link
Member

@lscheibel @karsa-mistmere Nice, I've one suggestion I'm curious what you think of it.
Maybe rename this to tree-broadleaf and create add another icon for tree-pine. I can image sometimes the pine tree is for some people more obvious choice.

@karsa-mistmere
Copy link
Member

@ericfennis Yeah, totally agree, there should be different kinds of trees. There's actually already an issue in progress: #599
I would maybe name this tree-deciduous (but it definitely needs it at least as a tag).

Here's a mashup overview of current design proposals by @lscheibel and myself.
image
tree

@ericfennis
Copy link
Member

@karsa-mistmere I like the top row. But with the current tree-deciduous icon.

@karsa-mistmere
Copy link
Member

karsa-mistmere commented Jun 1, 2022

@ericfennis: I guess that can work, but I actually quite grew to like the top row version, it's quite unmistakable and I think @lscheibel is right that the second row version does in fact look a bit like cotton candy.

In fact, I think we could maybe also add @lscheibel's original design as shrub. 😁

@ericfennis
Copy link
Member

Yeah that's true, maybe keep the branches.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants