peterfajdiga / karousel

KWin scrollable tiling script
GNU General Public License v3.0
195 stars 3 forks source link

Window detach if drag and dropped to another virtual desktop using the Pager applet #52

Open e455a81e-d3ba-41a2-bc6d-7aafb1d9a5cd opened 1 week ago

e455a81e-d3ba-41a2-bc6d-7aafb1d9a5cd commented 1 week ago

Hello I just installed Karousel, everything seems to work great. Impressive work! I just noticed a small bug.

Reproduction:

  1. Add the Pager virtual desktop switcher applet to a panel.
  2. Install Karousel via the store in the Plasma system settings
  3. Open a few Windows and tile them with karousel
  4. In the Pager applet grab a window and drag it to another desktop
  5. Switch to the desktop where you dragged the window and observe that the window is now no longer tiled

System: Operating System: Fedora Linux 40 (Kinoite) KDE Plasma Version: 6.1.0 KDE Frameworks Version: 6.3.0 Qt Version: 6.7.1 Kernel Version: 6.9.5-200.fc40.x86_64 (64-bit) Graphics Platform: Wayland

peterfajdiga commented 5 days ago

Thanks for the report! Sounds possibly related to https://github.com/peterfajdiga/karousel/issues/51. I'll check it out when I find the time to try Plasma 6.1.

peterfajdiga commented 18 hours ago

It's not related to https://github.com/peterfajdiga/karousel/issues/51.

The problem is that for some reason, the Pager temporarily puts the window on both desktops, which is not supported by Karousel. Implementing support for windows on multiple desktops would solve this issue, but it would be a big task and it's currently not planned.