D5490: Only follow mouse when moved (Fixes Bug #372635)

Leszek Lesner noreply at phabricator.kde.org
Tue Apr 18 12:35:05 UTC 2017


leszeklesner added a comment.


  In https://phabricator.kde.org/D5490#103049, @davidedmundson wrote:
  
  > Yes, and you have this boolean separately for each delegate.
  >
  > So I can see how this supresses that initial selection in your bug report; but surely it will also supress every other normal case after that.
  >
  > When we move from one delegate to another the first time from the point of view of the delegate we're moving into:
  >
  >   you get an onContainsMouseChanged -  _moved is falsed. We set it to true, but otherwise do nothing
  >   we get a onPositionChanged - we set _moved to true
  >   
  >
  > We're not going to get another onContainsMouseChanged unless there's a bug in Qt.
  
  
  Yeah you are right. Upon testing I noticed that I need to hover twice over an item to actually highlight it. 
  Hmm... so we need to place the _moved outside the delegate that should do it

REPOSITORY
  R112 Milou

REVISION DETAIL
  https://phabricator.kde.org/D5490

To: leszeklesner, broulik
Cc: davidedmundson, 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/742af1fc/attachment.html>


More information about the Plasma-devel mailing list