Vencord / Vesktop

Vesktop is a custom Discord App aiming to give you better performance and improve linux support
GNU General Public License v3.0
3.5k stars 168 forks source link

Ensure the .desktop file properly groups with created windows #305

Closed Exponential-Workload closed 8 months ago

Exponential-Workload commented 8 months ago

Currently, on Linux, when you pin the Vencord Desktop .desktop entry to your taskbar, and launch Vencord, you get this: image Ideally, it would instead group together with the pinned icon, as intended.

This issue can be fixed by making the .desktop file contain the following:

StartupWMClass=vencorddesktop

Currently, atleast on the AUR package for Vencord (vencord-desktop-git), this line reads StartupWMClass=vencord-desktop, which does not match the window's WM_CLASS(STRING) = "vencorddesktop", "VencordDesktop"; the - is causing this bug.

lewisakura commented 8 months ago

this is already the case on our end https://github.com/Vencord/Vesktop/blob/1429815fd126d802235b1dba041d6509ea6cd312/package.json#L114 the AUR package (which we don't maintain, it is by @zt64) has an outdated desktop file

Exponential-Workload commented 8 months ago

ah alright, my bad - didn't know where to report the bug

Vendicated commented 8 months ago

https://aur.archlinux.org/packages/vencord-desktop-git is the correct place for discussion of the aur package. or you could talk to the author on our discord server or email them

alphabitserial commented 7 months ago

@Exponential-Workload I've just left a comment on the AUR page, hopefully the maintainer updates their package soon. :smile:

Exponential-Workload commented 7 months ago

@Exponential-Workload I've just left a comment on the AUR page, hopefully the maintainer updates their package soon. :smile:

i left one too a day after making this issue, still no update - atp im tempted to just make a fork of the pkg (I'm using -git)

not sure if the pkg is considerable as abandoned yet or not

zt64 commented 7 months ago

@Exponential-Workload Now it should be fixed Also I just noticed Vesktop is on the chaotic-aur, so if you're using that it'll be some time before those get rebuilt