Bug 454584 - Overview effect should move dragged window based on the mouse position, not the center of the window.
Summary: Overview effect should move dragged window based on the mouse position, not t...
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: effects-overview (other bugs)
Version First Reported In: 5.24.4
Platform: Kubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-05-29 20:11 UTC by xojoyac741
Modified: 2022-06-07 22:36 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In: 5.25
Sentry Crash Report:


Attachments
Video describing the bug. (3.57 MB, video/x-matroska)
2022-05-29 20:11 UTC, xojoyac741
Details

Note You need to log in before you can comment on or make changes to this 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