Skip to content

Fix desktop icon association in GNOME#41

Merged
freakboy3742 merged 1 commit into
beeware:mainfrom
bbusenius:fix-desktop-icon-association
Oct 24, 2025
Merged

Fix desktop icon association in GNOME#41
freakboy3742 merged 1 commit into
beeware:mainfrom
bbusenius:fix-desktop-icon-association

Conversation

@bbusenius

Copy link
Copy Markdown
Contributor

This brings the Linux system template in line with the AppImage template, which received this fix in beeware/briefcase-linux-appimage-template#10.

Without StartupWMClass in the .desktop file, the application icon does not appear when the program is running in GNOME. This same thing happened to Signal Desktop on my system and when I saw signalapp/Signal-Desktop#7515, I realized we could fix it here too.

This PR adds StartupWMClass={{ cookiecutter.app_name }} to the desktop file template. Testing on Ubuntu 24.04 with GNOME 46 confirmed that {{ cookiecutter.app_name }} (e.g., my-app) works correctly, while {{ cookiecutter.formal_name }} (e.g., My App) does not. I'm not sure why we need app_name instead of formal_name but this works on my system.

PR Checklist:

  • All new features have been tested
  • All new features have been documented
  • I have read the CONTRIBUTING.md file
  • I will abide by the code of conduct

@freakboy3742 freakboy3742 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix - definitely an oversight, and explains the odd behavior of system icons not appearing... given I thought we'd fixed that problem :-)

@freakboy3742
freakboy3742 merged commit d3453fc into beeware:main Oct 24, 2025
45 checks passed
@bbusenius

Copy link
Copy Markdown
Contributor Author

No problem! Thanks for the great software! I used it to build personal apps that I use daily. It's the most useful thing ever.

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.

2 participants