| hein |
| Plasma |
We don't actually process hoverEnter/move events, so there's no need to
have the overhead of having them sent to use.
We still get dragMove events regardless.
Setting it enabled broke FolderView nested in a grouping Plasmoid and
doesn't do anything useful.
Tested folderview in a grouping plasmoid
Also tested dragging an item from dolphin over an existing folderview.
Dragging over folders still worked as before
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
"Setting it enabled broke FolderView nested in a grouping Plasmoid" is this worth adding an auto test somewhere?
Maybe, but it'd be testing that an item doesn't do what it had no reason to do in the first place.
There's an infinite number of random things this item shouldn't do, I don't see what we'd gain from testing this specific one.
It doesn't even look like it was introduced deliberately: 438b6485a481e4c11a3e719ff65adb0c631b96ad
The FV side does rely on onDragMove firing, which is understand it still does regardless, so +1.