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

Archeological site 11 #13

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Archeological site 11 #13

wants to merge 2 commits into from

Conversation

valleyofdawn
Copy link
Owner

No description provided.

@natslaughter
Copy link

natslaughter commented Dec 5, 2016

Hello @valleyofdawn
Thank you for your icon submissions!

archeological-site icon feedback:
Overall, I think your icon concept works, since a 'vase' is globally recognized, however at 100% scale, the vase's silhouette might be too generic of a shape. If you have any other icon concepts that are both global and unique in shape, please consider them.

archeological-site-15 1

archeological-site-11 1

Your 11-pixel icon is not at the correct size

archeological-site-11 2

Pixel alignment issues

archeological-site-11 5

@valleyofdawn
Copy link
Owner Author

valleyofdawn commented Dec 5, 2016 via email

@natslaughter
Copy link

@valleyofdawn
Adding a handle sounds good.

@valleyofdawn
Copy link
Owner Author

valleyofdawn commented Dec 6, 2016 via email

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