D20838: Allow to drop one file or one folder on KDirOperator

Nathaniel Graham noreply at phabricator.kde.org
Fri Apr 26 17:15:21 BST 2019


ngraham added a comment.


  Wow, it works and it's sooooooooo nice. What an improvement!
  
  Drop options are not needed here; this isn't a move/copy/symlink UI; it's simply a quick way to choose a file that you happen to have visible in Dolphin or on the desktop. On that subject, I notice that drops from the desktop don't work. Not sure if that needs work here or in Folder View.

INLINE COMMENTS

> kdiroperator.cpp:1410
> +                if (entry.isDir()) {
> +                    // if this was a directory
> +                    setUrl(url, false);

Comment seems unnecessary; the code is clear enough

REPOSITORY
  R241 KIO

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

To: meven, ngraham
Cc: apol, kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190426/70d91262/attachment.html>


More information about the Kde-frameworks-devel mailing list