Review Request: Patch for Wish 300578 - [Idea] Multi paste feature

Frank Reininghaus frank78ac at googlemail.com
Sat Aug 4 08:04:30 BST 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105719/#review16847
-----------------------------------------------------------


Thanks for the patch! Code looks good in principle, but I'm not quite sure if the "usefulness vs. possible confusion" ratio of this feature is good enough to justify its inclusion.

I think I never wanted to paste something in multiple sub-folders of the current folder, and I can't think of any common situation when this could be necessary. 

- Frank Reininghaus


On July 24, 2012, 8:11 p.m., Emmanuel Pescosta wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105719/
> -----------------------------------------------------------
> 
> (Updated July 24, 2012, 8:11 p.m.)
> 
> 
> Review request for Dolphin and Frank Reininghaus.
> 
> 
> Description
> -------
> 
> With this patch, you can paste files to all selected folders in one go.
> 
> 
> This addresses bug 300578.
>     http://bugs.kde.org/show_bug.cgi?id=300578
> 
> 
> Diffs
> -----
> 
>   dolphin/src/dolphincontextmenu.cpp bb26c7a 
>   dolphin/src/views/dolphinview.cpp 233c700 
> 
> Diff: http://git.reviewboard.kde.org/r/105719/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Emmanuel Pescosta
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20120804/12058949/attachment.htm>


More information about the kfm-devel mailing list