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 icon for the Decal node #37862

Merged
merged 1 commit into from
Apr 24, 2020
Merged

Conversation

YeldhamDev
Copy link
Member

Very simple design, but I think it gets the point across.

Depends on #37861 being merged first, of course.

@clayjohn
Copy link
Member

Oooh, I really like it! Something like this would be really nice for the new *SkyMaterials. If you are interested in making icons, PanoramaSkyMaterial ProceduralSkyMaterial and PhysicalSkyMaterial all need some.

@reduz
Copy link
Member

reduz commented Apr 14, 2020

I have to admit its nice, but it took me a while to figure out what it is... I was thinking a stamp might be more obvious.
image

@akien-mga
Copy link
Member

I like the "sticker" design.

The stamp is also nice, but my first thought was that it was a chess pawn, so I'm not sure it will be clear for everyone.

Before seeing any proposal, my idea was an icon that would be like paint splatter/splash, but it might not work well in our small icon size.

@YeldhamDev
Copy link
Member Author

YeldhamDev commented Apr 14, 2020

Just reminding that icon nodes are presented very small, so the less detail they have, the better.

@RibalGZ
Copy link

RibalGZ commented Apr 14, 2020

I think the stamp is not a good idea, it's commonly used for cloning tools.

@akien-mga akien-mga merged commit 12c36e2 into godotengine:master Apr 24, 2020
@akien-mga
Copy link
Member

Thanks!

@YeldhamDev YeldhamDev deleted the decal_icon branch April 24, 2020 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants