D5490: Only follow mouse when moved (Fixes Bug #372635)
Leszek Lesner
noreply at phabricator.kde.org
Tue Apr 18 10:43:57 UTC 2017
leszeklesner created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
Use a new variable __moved to detect if mouse moved and only change index if the mouse moved. This helps preventing index changes when only using keyboard to search something in milou and to not accidently start/open something you did not want (see bug report https://bugs.kde.org/show_bug.cgi?id=372635 )
In general the onEntered signal seems to be broken in Qt somehow as I could not make it work reliably with Qt 5.7.1. Sometimes it worked but mostly the code using onEntered failed to work with onPositionChanged (I guess this also does not always set it to true).
Hence I tried containsMouse which seems to work reliably at least on Qt 5.7.1.
Tests are appreciated especially on other Qt versions.
REPOSITORY
R112 Milou
REVISION DETAIL
https://phabricator.kde.org/D5490
AFFECTED FILES
lib/qml/ResultDelegate.qml
To: leszeklesner, broulik
Cc: plasma-devel, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170418/ddae0a05/attachment.html>
More information about the Plasma-devel
mailing list