Review Request 125118: [Icon widget] support dropping onto folder

Kai Uwe Broulik kde at privat.broulik.de
Wed Sep 9 19:14:27 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125118/
-----------------------------------------------------------

Review request for Plasma, Bhushan Shah, Emmanuel Pescosta, and Eike Hein.


Repository: plasma-workspace


Description
-------

This allows to drop file(s) onto a folder and have it ask whether to copy/move/link and brings up error dialogs if any.


Diffs
-----

  applets/icon/package/contents/ui/main.qml 49e91ca 
  applets/icon/plugin/icon_p.h 5b0f752 
  applets/icon/plugin/icon_p.cpp 97af67a 

Diff: https://git.reviewboard.kde.org/r/125118/diff/


Testing
-------

Dropping brings up the copy/move/paste dialog, copied, moved, and linked files, works. Confirmation dialogs are desktop modal. Also, I had to create my own QDropEvent from the declarative one.

Modifiers are broken but this is apparently a Qt bug, also I don't know whether my int cast to flags is right. Thanks Emmanuel for guiding me through this.


Thanks,

Kai Uwe Broulik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150909/7095084d/attachment-0001.html>


More information about the Plasma-devel mailing list