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

Add StartupWMClass on Linux desktop file #1138

Closed
wants to merge 1 commit into from
Closed

Add StartupWMClass on Linux desktop file #1138

wants to merge 1 commit into from

Conversation

bilelmoussaoui
Copy link
Contributor

StartupWMClass is used to identify the opened applications under Linux with their desktop files, which means using the right icon and so on... more information can be found here https://standards.freedesktop.org/desktop-entry-spec/latest/ar01s05.html

The value assigned on StartupWMClass can be easily found using xprop WM_CLASS and click other application you would like to get the WM_CLASS.

What does this affect? Nothing at all, The only difference is that if an icon theme includes an icon for WebTorrent it will be used instead of using the default one. Also, the default one will look blurry on some Desktop environments. And the user won't be able to get to use the actions defined in the desktop file while running the application if it's not correctly identified. This is mostly needed for non-native applications

Without a WM_CLASS:
screenshot from 2017-02-16 18-53-18

With a WM_CLASS:
screenshot from 2017-02-16 18-55-01

@stale
Copy link

stale bot commented May 10, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the stale label May 10, 2018
@feross feross reopened this May 20, 2018
@stale stale bot removed the stale label May 20, 2018
Copy link
Contributor

@mathiasvr mathiasvr left a comment

Choose a reason for hiding this comment

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

I have not tested it, but I will approve based on some google searches. 😃Seems legit.

@bilelmoussaoui
Copy link
Contributor Author

Any news?

@hicom150
Copy link
Contributor

hicom150 commented Sep 9, 2019

LGTM 👍

@feross
Copy link
Member

feross commented Sep 10, 2019

@hicom150 Is this change incorporated into your PR #1682 ? If so, can this be closed once that is merged?

@hicom150
Copy link
Contributor

Is this change incorporated into your PR #1682

Yes it is already in my PR, so we can close this one 😉

BTW, thank you @bilelmoussaoui for this PR 👍

@feross
Copy link
Member

feross commented Sep 10, 2019

Cool, closing this. Thanks @bilelmoussaoui!

@feross feross closed this Sep 10, 2019
@bilelmoussaoui bilelmoussaoui deleted the patch-1 branch September 10, 2019 21:15
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.

4 participants