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

Structure icons (bikeshedding) #65

Closed
Treer opened this issue Feb 14, 2016 · 5 comments
Closed

Structure icons (bikeshedding) #65

Treer opened this issue Feb 14, 2016 · 5 comments
Milestone

Comments

@Treer
Copy link
Collaborator

Treer commented Feb 14, 2016

In AmidstExporter I have lightened the NetherFortress icon, we may want to do the same:

nether_fortress original -> nether_fortress

And tried to come up with a better witchhut icon, I don't like the original icon:
witch
but it's hard to do better as the witch hat makes a head too big to draw a witch the same way village icons are drawn.

witchhut1
witchhut2 - many of the other structure icons just show the primary material
witchhut3 - but a witchhut has a primary material and produces witches.
witchhut4 - actually they are built from both planks and logs.
witchhut5 - back to logs, but with a lightened witch
witchhut6 - hat made darker

I have used the last one in AmidstExporter bcause it's more obvious to people what it is (e.g. when screenshots are posted online) but it doesn't fit the Amidst style very well.

So if anyone wants to try improving witch hut icons...

@Treer Treer changed the title structure icons (bikeshedding) Structure icons (bikeshedding) Feb 14, 2016
@BigAlanM
Copy link
Contributor

I have no problem with the new icons.
Actually I've never cared for the the dark icons.
Yours are a definite improvement.

@stefandollase
Copy link
Contributor

I also do like the new icons. I think we should take the new nether fortress icon for sure. For the Witch Hut, I prefer the cauldron.

@Treer
Copy link
Collaborator Author

Treer commented Feb 15, 2016

witchhut7

@stefandollase
Copy link
Contributor

I like it :-)

@Treer Treer mentioned this issue Feb 20, 2016
@stefandollase stefandollase added this to the v4.0 milestone Feb 21, 2016
@stefandollase
Copy link
Contributor

I am closing this, since the new icons are included in the new release.

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

No branches or pull requests

3 participants