Skip to content

Commit

Permalink
macOS App Icon fix
Browse files Browse the repository at this point in the history
• Apple UI guidelines adherent Finder icon and Dock icon
• Delete now unnecessary Image.xcassets directory and its content
• Required changes in Theengs.pro and main.cpp
• New Icon with wider margins
  • Loading branch information
DigiH committed Jun 20, 2022
1 parent 45f28e1 commit 1e438b5
Show file tree
Hide file tree
Showing 11 changed files with 2 additions and 4 deletions.
3 changes: 1 addition & 2 deletions Theengs.pro
Original file line number Diff line number Diff line change
Expand Up @@ -267,8 +267,7 @@ macx {
QMAKE_BUNDLE = app

# OS
QMAKE_ASSET_CATALOGS_APP_ICON = "AppIcon"
QMAKE_ASSET_CATALOGS = $${PWD}/assets/macos/Images.xcassets
ICON = $${PWD}/assets/macos/theengs.icns

# OS infos
QMAKE_INFO_PLIST = $${PWD}/assets/macos/Info.plist
Expand Down
Binary file not shown.
Binary file not shown.
Binary file removed assets/macos/Image.xcassets/AppIcon.appiconset/16.png
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.

This file was deleted.

Binary file modified assets/macos/theengs.icns
Binary file not shown.
2 changes: 1 addition & 1 deletion src/main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ int main(int argc, char *argv[])
app.setOrganizationName("Theengs");
app.setOrganizationDomain("Theengs");

#if !defined(Q_OS_ANDROID) && !defined(Q_OS_IOS)
#if !defined(Q_OS_ANDROID) && !defined(Q_OS_IOS) && !defined(Q_OS_MACOS)
// Application icon
QIcon appIcon(":/assets/logos/logo.png");
app.setWindowIcon(appIcon);
Expand Down

0 comments on commit 1e438b5

Please sign in to comment.