D5573: Don't accept hover events in the DragArea

David Edmundson noreply at phabricator.kde.org
Tue Apr 25 15:33:50 UTC 2017


davidedmundson added a comment.


  In https://phabricator.kde.org/D5573#104619, @aacid wrote:
  
  > "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: https://phabricator.kde.org/R296:438b6485a481e4c11a3e719ff65adb0c631b96ad

REPOSITORY
  R296 KDeclarative

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

To: davidedmundson, #plasma
Cc: aacid, plasma-devel, #frameworks, spstarr, progwolff, Zren, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170425/db9b5c64/attachment.html>


More information about the Plasma-devel mailing list