Bug 500236

Summary: Drag windows from desktop to another inside the pager resets early
Product: [Plasma] plasmashell Reporter: JR <jr>
Component: Pager widgetAssignee: Plasma Bugs List <plasma-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: akselmo, hein, nate
Priority: NOR Keywords: regression
Version First Reported In: 6.3.0   
Target Milestone: 1.0   
Platform: openSUSE   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 6.3.2
Sentry Crash Report:
Attachments: Screencast of pager usage.

Description JR 2025-02-17 09:41:01 UTC
Created attachment 178469 [details]
Screencast of pager usage.

SUMMARY
See the screen capture appended, moving window frames from a virtual desktop to another does not work anymore.

STEPS TO REPRODUCE
1. upgrade kde plasma to 6.3.0
2. Restart system which restores the last open windows (on the wrong virtual desktop, but ok)
3. Trying to move the windows back to the correct virtual desktops, it does not let me do this anymore. 

OBSERVED RESULT
See video.

EXPECTED RESULT
Moving windows between virtual desktops is an important task in my daily workflow. It plasma5 at the end it works fine. Since 6 I fear every update because it seems to have regressions in many updates.

SOFTWARE/OS VERSIONS
Operating System: openSUSE Tumbleweed 20250211
KDE Plasma Version: 6.3.0
KDE Frameworks Version: 6.10.0
Qt Version: 6.8.2
Kernel Version: 6.13.1-1-default (64-bit)
Graphics Platform: Wayland
Processors: 16 × AMD Ryzen 7 PRO 5750G with Radeon Graphics
Memory: 29,2 GiB of RAM
Graphics Processor: AMD Radeon Graphics

ADDITIONAL INFORMATION
Comment 1 Akseli Lahtinen 2025-02-18 13:07:42 UTC
Can confirm this issue. 

Operating System: Fedora Linux 41
KDE Plasma Version: 6.3.80
KDE Frameworks Version: 6.12.0
Qt Version: 6.8.2
Kernel Version: 6.12.13-200.fc41.x86_64 (64-bit)
Graphics Platform: Wayland
Processors: 12 × AMD Ryzen 5 3600 6-Core Processor
Memory: 15.5 GiB of RAM
Graphics Processor: AMD Radeon RX 6600
Comment 2 Bug Janitor Service 2025-02-18 14:12:31 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/2831
Comment 3 Akseli Lahtinen 2025-02-18 14:33:15 UTC
Git commit 2062574e87f2ecf79daa70048fc737abd6aa460f by Akseli Lahtinen.
Committed on 18/02/2025 at 14:09.
Pushed by akselmo into branch 'master'.

pager: remove onContainsMouseChanged, refresh model after desktopMouseArea.clicked

Due to refreshing the pager twice on mouse release and having two
different mouse drag tracking areas, these could get out of sync.
This caused the pagerModel refresh too early and lose any changes.
This changes it to only refresh after mouse is released, either when
dragging a window or clicking a desktop.

M  +1    -10   applets/pager/package/contents/ui/main.qml

https://invent.kde.org/plasma/plasma-desktop/-/commit/2062574e87f2ecf79daa70048fc737abd6aa460f
Comment 4 Akseli Lahtinen 2025-02-18 14:50:56 UTC
Git commit 4c162cf94163453bcc89679d25281484932dcfce by Akseli Lahtinen.
Committed on 18/02/2025 at 14:33.
Pushed by akselmo into branch 'Plasma/6.3'.

pager: remove onContainsMouseChanged, refresh model after desktopMouseArea.clicked

Due to refreshing the pager twice on mouse release and having two
different mouse drag tracking areas, these could get out of sync.
This caused the pagerModel refresh too early and lose any changes.
This changes it to only refresh after mouse is released, either when
dragging a window or clicking a desktop.


(cherry picked from commit 2062574e87f2ecf79daa70048fc737abd6aa460f)

Co-authored-by: Akseli Lahtinen <akselmo@akselmo.dev>

M  +1    -10   applets/pager/package/contents/ui/main.qml

https://invent.kde.org/plasma/plasma-desktop/-/commit/4c162cf94163453bcc89679d25281484932dcfce