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

Changed godot.svg icon #2723

Merged
merged 3 commits into from
Jun 21, 2021
Merged

Changed godot.svg icon #2723

merged 3 commits into from
Jun 21, 2021

Conversation

liconaj
Copy link
Contributor

@liconaj liconaj commented Jun 21, 2021

This is an attemp to improve the appearance of the Godot game engine icon because, in my opinion, it was painful to see. I tried to adjust most elements to the grid, and it was quite hard (old icon wins in here). Nevertheless, I think that this new design is good enough: colors are the same, most of the elements are adjusted, shape more similar compared with the original logo, and padding and 'shadows' are respected. Hope it is worth it.

Current icon (64x64)
old-godot@64x64

New icon (64x64)
godot@64x64

@varlesh
Copy link
Contributor

varlesh commented Jun 21, 2021

thx for PR, but you not use pixel align (it's our HIG). Compare with Papirus original icon (look at the teeth). Your lines blured.
I think better solution - change only eyes and the main background

@SmartFinn SmartFinn merged commit d32cb4f into PapirusDevelopmentTeam:master Jun 21, 2021
@liconaj
Copy link
Contributor Author

liconaj commented Jun 21, 2021

Thank you so much for your advice. Now it's supposed to be all pixel perfect right now. Do I have to make another PR if they are okay, btw?

Here are how they look like:

16x16
godot@16x16

22x22
godot@22x22

24x24
godot@24x24

32x32
godot@32x32

48x48
godot@48x48

64x64
godot@64x64

SmartFinn added a commit that referenced this pull request Jun 21, 2021
- replace fill-opacity with opacity
- fix vertical alignment
@liconaj
Copy link
Contributor Author

liconaj commented Jun 21, 2021

Hahaha, Now I see that you have already emerged it. Thanks!

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