Operating System: KDE neon User Edition 5.10 Short description: ------------------- Since updating to Plasma 5.10 switching the desktop when moving a desktop to an edge does not work anymore. This bug appears with [System Settings -> Desktop Behavior -> Screen Edges -> Switch desktop on edge] set to "Only When Moving Windows". Steps to reproduce: ------------------ 1. Set [System Settings -> Desktop Behavior -> Screen Edges -> Switch desktop on edge] to "Only When Moving Windows" 2. Update to Plasma 5.10 3. Drag a window (either by the titlebar or with Alt+Leftclick) against a screen edge. Expected Behavior: -------------------- A desktop switch should be triggered. Actual Behavior: ---------------- Nothing happens.
I see the same issue on both Kubuntu and KDE Neon 5.10 packages.
*** Bug 380959 has been marked as a duplicate of this bug. ***
*** Bug 381139 has been marked as a duplicate of this bug. ***
Maybe related to bug 380476.
(In reply to Christoph Feck from comment #4) > Maybe related to bug 380476. unlikely. The code path for moving the windows to different vd is different and doesn't use the windows at all.
Issue can be reproduced on both X11 and Wayland. I have an idea of what went wrong.
Created attachment 106145 [details] Possible patch Attached is a possible patch for the problem. I'm also going to try to create an autotest for it, so that it cannot regress again.
Patch with autotest at: https://phabricator.kde.org/D6257
Git commit c45e165514ec70314d721d6e0238431ab92bd441 by Martin Flöser. Committed on 20/06/2017 at 04:57. Pushed by graesslin into branch 'Plasma/5.10'. Fix switch desktop through edge when moving window Summary: There was a regression introduced in ScreenEdges when introducing the activatesForPointer method. It considered the switch desktop on edge, but not the special case of switch desktop when moving windows. Due to that the edges did not activate when moving the window. This change addresses the regression and extends the autotest to ensure it's properly covered. FIXED-IN: 5.10.3 Test Plan: Manual testing and extended auto test Reviewers: #kwin, #plasma Subscribers: plasma-devel, kwin Tags: #kwin Differential Revision: https://phabricator.kde.org/D6257 M +11 -0 autotests/mock_abstract_client.cpp M +3 -0 autotests/mock_abstract_client.h M +0 -5 autotests/mock_client.cpp M +0 -2 autotests/mock_client.h M +21 -0 autotests/test_screen_edges.cpp M +6 -0 screenedge.cpp https://commits.kde.org/kwin/c45e165514ec70314d721d6e0238431ab92bd441