minecraft-linux / mcpelauncher-appimage

4 stars 1 forks source link

Either the launcher or the game is not recognized #3

Open skewty opened 4 years ago

skewty commented 4 years ago

When I run MC.AppImage I see more than one entry on my dock. This means the window class isn't being recognized.

mcpelauncher.desktop

One of the following lines should also be in the .desktop file

# Recognizes the game window
StartupWMClass=mcpelauncher-client
# Recognizes the launcher window
StartupWMClass=AppRun.wrapped

Preferably both the game and launcher would use the same window class somehow.

ChristopherHX commented 4 years ago

I guess you know more about x11 than me.

glfw seems to allow a custom wmclass qt5 uses argv[0] as it's wmclass qt5 mcpelauncher-ui-qt -name mcpelauncher -name cli option

Also applies to filepicker https://github.com/minecraft-linux/file-picker/blob/54b36967c46fa2e0baf273a422ea010dbe5841db/src/file_picker_zenity.cpp#L13 Need to add the --class switch https://askubuntu.com/questions/480783/why-does-a-bash-zenity-script-has-that-title-on-unity-panel-and-that-icon-on-uni

That change won't happen soon unless by a PR.

ChristopherHX commented 3 years ago

Partially done in ng.AppImage (https://github.com/ChristopherHX/linux-packaging-scripts/releases/tag/ng.appimage) Works after installing the app via the appimagelauncher.

The msa-ui-qt window still has the wrong x11 class.