flameshot icon indicating copy to clipboard operation
flameshot copied to clipboard

Add generic wlroots DesktopInfo for compositors such as labwc

Open joshuataylor opened this issue 2 years ago • 4 comments

Compositors such as labwc use "wlroots" as their XDG_CURRENT_DESKTOP.

I like that they do that, and hope others do it as well, as it means Flameshot doesn't need to add an ∞ amount of case statements.

joshuataylor avatar Apr 02 '23 08:04 joshuataylor

isn't Sway a wlroot? I'm also not sure about hyprland but I think that is also wlroot. If these are correct, then perhaps having them in src/utils/desktopinfo.h and src/utils/screengrabber.cpp is redundant.

mmahmoudian avatar Apr 02 '23 12:04 mmahmoudian

isn't Sway a wlroot? I'm also not sure about hyprland but I think that is also wlroot. If these are correct, then perhaps having them in src/utils/desktopinfo.h and src/utils/screengrabber.cpp is redundant.

sway self reports as sway

✦2 ❯ echo $XDG_CURRENT_DESKTOP
sway

xvkdev avatar Apr 14 '23 21:04 xvkdev

Hyprland reports itself at Hyprland

✦ ❯  echo $XDG_CURRENT_DESKTOP
Hyprland

RichAyotte avatar Sep 03 '23 16:09 RichAyotte

Is there utility in this given sway reports as "sway" and hyrland reports as "hyprland"?

borgmanJeremy avatar May 29 '25 00:05 borgmanJeremy