trmdi added a comment.
In D16643#354120 <https://phabricator.kde.org/D16643#354120>, @bruns wrote: > `m_enabled` may change between //consecutive// DragMove events, but not during an event. > > In this case, though, it does not, and is constantly true. You can verify this by attaching a debugger or by adding debug statements. > > The only wrong code part is the `setAccepted(false)`, which should be `setAccepted(true)` (or just `accept()`, both are equivalent). Please see these videos, note the cursor icon: - If I place `setAccept(true)` below the first if like you said: https://streamable.com/oj1j4 -> the cursor does not change - If I place it above the first if like I've done in my patch: https://streamable.com/2g58g -> the cursor changes REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D16643 To: trmdi, mart, broulik, #plasma, hein, davidedmundson Cc: bruns, dkorth, ngraham, kde-frameworks-devel, michaelh
