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

Adds leaf cross section #161

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

XylotrupesGideon
Copy link
Contributor

Adds the leaf cross section from #79

@duerrsimon
Copy link
Owner

Might be more appropriate in the plant category? What do you think?

@XylotrupesGideon
Copy link
Contributor Author

Hmm I think it is a classical case of double category.
I thought the plant section would be used for icons of full plants in the same way as the animal section has full animals and then the different tissues of these organisms go to the tissue category. I think the animal category would get very crowded if we add icons of all different tissues and organs over time.
That's why I decided on the tissue category here...
Maybe an explicit description of what goes into which category would be useful for contributers

@duerrsimon
Copy link
Owner

Also here the comment, that multi categories are coming!

For now I would still put it in plants since the two other cross sections are also in plants. I agree with your reasoning but best to be consistent for now.

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.

2 participants