search

bojidar-bg / plasma-parallax-wallpaper

A Plasma 5 plugin which moves a background image based on the current desktop
14 stars 1 forks source link

Broken on Wayland on Plasma 5.27.3 #5

Closed hifinerd closed 8 months ago

hifinerd commented 1 year ago

Title. This plugin does not scroll the wallpaper on Wayland, it just stays still. However, it still works fine on X11. I am using Arch Linux with an AMD iGPU.

bojidar-bg commented 1 year ago

Unfortunately, I don't have a wayland install I could test this on at the moment. Any patches would be welcome, however!

1e1001 commented 1 year ago

It looks like the issue is that KWindowSystem::currentDisplay() got deprecated (or wasn't implemented on Wayland in the first place), the documentation suggests using KX11Extras::currentDisplay() instead. Not sure what the Wayland version of the function is (if it even exists)

bojidar-bg commented 8 months ago

Accidentally fixed in https://github.com/bojidar-bg/plasma-parallax-wallpaper/commit/ed3431983ef42114dcef4ade3b47487c5a58a35b using TaskManager::VirtualDesktopInfo::currentDisplay()