-
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Desktop integration of app icon #5
Comments
Can you describe in more detail what exactly is not working? If the application sets the icon on the window, and if the desktop file uses the StartupWMClass field in the .desktop file correctly, then it should work sufficiently? |
In our case, we're avoiding the use of @TheAssassin/AppImageLauncher (as mentioned). With that in hand, the issues are:
Only thing I can request you to take care of is reducing the disconnect between 2 & 3. And of course, thanks for the awesome tool. Even with some limitations, it's still my favorite package distribution mechanism on *nix. |
Thanks @debloper. I agree that the current state is not ideal, but I am not sure what we could do from the AppImage side to improve things without the XDG specifications and/or desktop environments being changed. Please let me know if you have concrete ideas how to tackle the issues you've described. Thanks for using AppImage. |
As of now with
AppImage
&electron-builder
there's seems to be no reasonable way to make app icons work consistently across Linux DE. We are left with two choices:deb
,rpm
,pacman
or newersnap
,flatpak
etc.)At the moment, I'm personally against the idea of
#2
because:deb
,rpm
,pacman
etc.) aren't an exhaustive listWe can also debate the possibility of using a different packaging utility, but there isn't any good
electron-builder
alternative or a good way toSo, for now, we're stuck with these two tools, unless until something changes.
So, we're going ahead with Plan A:
A consolidated analysis to understand the issue with
#1
(thanks @probonopd):AppImageCommunity/appimaged#30
The text was updated successfully, but these errors were encountered: