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

Use a Big Sur-style macOS icon for the Godot editor #52043

Merged
merged 1 commit into from
Sep 30, 2021

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Aug 23, 2021

This provides a more coherent visual look with the rest of macOS.

This closes godotengine/godot-proposals#1616.

Thanks @hazarek for providing the icon 🙂

Preview

I don't have a macOS setup at hand right now, so here's Gwenview rendering it for now:

image

This provides a more coherent visual look with the rest of macOS.

Co-authored-by: Hazar <36481442+hazarek@users.noreply.github.com>
@Calinou Calinou requested a review from a team as a code owner August 23, 2021 23:53
@Calinou Calinou added cherrypick:3.x Considered for cherry-picking into a future 3.x release enhancement platform:macos topic:editor labels Aug 23, 2021
@Calinou Calinou added this to the 4.0 milestone Aug 23, 2021
@bruvzg
Copy link
Member

bruvzg commented Aug 24, 2021

Icon is overeiden during startup. Changing one in the bundle template won't fully change app icon, see #48685 for the relevant code (disables icon override), or the same code can be changed to use macOS specific icon.

@Calinou Calinou marked this pull request as draft August 24, 2021 04:37
@Calinou
Copy link
Member Author

Calinou commented Sep 30, 2021

It works successfully following the merge of #48685:

image

@Calinou Calinou marked this pull request as ready for review September 30, 2021 13:37
@akien-mga akien-mga merged commit 50bb089 into godotengine:master Sep 30, 2021
@akien-mga
Copy link
Member

Thanks!

@Calinou Calinou deleted the macos-use-big-sur-icon branch September 30, 2021 14:09
@akien-mga
Copy link
Member

Cherry-picked for 3.4.

@akien-mga akien-mga removed the cherrypick:3.x Considered for cherry-picking into a future 3.x release label Oct 4, 2021
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.

Big Sur compatible new macOS app icon proposal
3 participants