App launcher on Linux opens a new icon

[DireMunchkin]

Describe the bug
Since version 1.10 there was a change in the behavior of the app launcher. I used to be able to pin the app launcher and once the program is open I can click or use Super+<n> hotkey to switch to it. Now the app launcher opens a new icon, which cannot be pinned.

To Reproduce
Steps to reproduce the behavior:

1. Pin the Jellyfin launcher to task bar on Linux, w Cinnamon.
2. Click on it.
3. Minimize the app window.
4. Click on it again.

Expected behavior
I expect the window to be raised or focused, instead it does nothing.

Screenshots

Desktop (please complete the following information):

• OS: Linux Mint 21.3
• Version 1.10.1
• Installation Method: DEB

[7system7]

Same here.

OS: Arch Linux x86_64 
Kernel: 6.9.1-arch1-1 
DE: GNOME 46.1 
WM: Mutter 

[rafalb8]

This is caused by this change 7f72a3e.
Here StartupWMClass was updated, but running

xprop WM_CLASS

shows WM_CLASS(STRING) = "jellyfinmediaplayer", "Jellyfin Media Player"

[M4dMikel]

There is a pull request that should fix this Issue:
#771

(credit goes to https://github.com/solidsnake1298 for creating it)

[bjornfor]

This is caused by this change 7f72a3e.

@Fdmt3ch: On what system was that change needed? Your commit message says "Wayland", but I'm on GNOME with Wayland and and need your change (partly?) reverted for things to work again.

[Fdmt3ch]

That was using early stable of KDE Plasma 6.1.x at the time. If a change / revert is required to make things work on different Wayland compositors, go ahead 👍On 13 Jan 2025, at 21:14, Bjørn Forsman ***@***.***> wrote: This is caused by this change 7f72a3e. @Fdmt3ch: On what system was that change needed? Your commit message says "Wayland", but I'm on GNOME with Wayland and and need your change (partly?) reverted for things to work again. —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>

[bjornfor]

@Fdmt3ch: I think we need someone with commit rights to merge #771.

[Fdmt3ch]

Note, See PR #515

[bjornfor]

Note, See PR #515

Hm, maybe this isn't as straight forward as I first thought.