Bug 454584

Summary: Overview effect should move dragged window based on the mouse position, not the center of the window.
Product: [Plasma] kwin Reporter: xojoyac741 <xojoyac741>
Component: effects-overviewAssignee: KWin default assignee <kwin-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: dev.bacteriostat, nate
Priority: NOR    
Version First Reported In: 5.24.4   
Target Milestone: ---   
Platform: Kubuntu   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 5.25
Sentry Crash Report:
Attachments: Video describing the bug.

Description xojoyac741 2022-05-29 20:11:01 UTC
Created attachment 149321 [details]
Video describing the bug.

SUMMARY
***
When dragging a window, I would expect that the 'trigger' for changing/creating desktops would be the mouse pointer, not the center of the window. This could cause a problem where if you grab the window by the corners, even if you have your mouse pointer on desktop 3, the window will be moved to desktop 2.
***


STEPS TO REPRODUCE
1. Open Overview effect
2. Grab window by the corner
3. Move mouse over do "desktop 3"

OBSERVED RESULT
The window will be moved to desktop 2

EXPECTED RESULT
The window should be moved to desktop 3

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: Kubuntu 22.04
(available in About System)
KDE Plasma Version: 5.24.4
KDE Frameworks Version: 5.92.0
Qt Version: 5.15.3

ADDITIONAL INFORMATION
Comment 1 Bacteria 2022-05-30 08:01:24 UTC
Can reproduce

Operating System: Arch Linux
KDE Plasma Version: 5.24.90
KDE Frameworks Version: 5.94.0
Qt Version: 5.15.4
Graphics Platform: Wayland
Comment 2 Bug Janitor Service 2022-05-31 14:46:06 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/2489
Comment 3 Marco Martin 2022-06-06 11:39:33 UTC
Git commit 6abce1529a33656ff7b9fb1264d3cca2ed1c062d by Marco Martin.
Committed on 06/06/2022 at 11:39.
Pushed by mart into branch 'master'.

Put the hostpot as the press position

we should drop the window on the desktop under the cursor,
not under the center of the thumbnail

M  +3    -1    src/effects/private/qml/WindowHeap.qml

https://invent.kde.org/plasma/kwin/commit/6abce1529a33656ff7b9fb1264d3cca2ed1c062d
Comment 4 Marco Martin 2022-06-06 11:42:54 UTC
Git commit 2f58c5bcf3bd3de29d4d6a68644825e480e3c2b6 by Marco Martin.
Committed on 06/06/2022 at 11:40.
Pushed by mart into branch 'Plasma/5.25'.

Put the hostpot as the press position

we should drop the window on the desktop under the cursor,
not under the center of the thumbnail

M  +3    -1    src/effects/private/qml/WindowHeap.qml

https://invent.kde.org/plasma/kwin/commit/2f58c5bcf3bd3de29d4d6a68644825e480e3c2b6