Review Request 115357: Add a shortcut for copying current directory

[Frank Reininghaus]

> On Jan. 28, 2014, 3:58 p.m., Frank Reininghaus wrote:
> > First of all, thanks for the patch!
> > 
> > To quote my comment https://bugs.kde.org/show_bug.cgi?id=323960#c3 :
> > 
> > "...this is one of the cases where I wonder if this feature is really useful enough that it justifies adding code which must be maintained in the future, and adding another action that makes the shortcuts dialog harder to use (I admit that it's a big mess not not very useable already)."

Hm, I shouldn't have copied the typo from the comment ("not not" -> "and not"):

"...this is one of the cases where I wonder if this feature is really useful enough that it justifies adding code which must be maintained in the future, and adding another action that makes the shortcuts dialog harder to use (I admit that it's a big mess and not very useable already)."


- Frank


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


On Jan. 28, 2014, 3:55 p.m., Yichao Zhou wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115357/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2014, 3:55 p.m.)
> 
> 
> Review request for Dolphin.
> 
> 
> Bugs: 323960
>     http://bugs.kde.org/show_bug.cgi?id=323960
> 
> 
> Repository: kde-baseapps
> 
> 
> Description
> -------
> 
> Add 2 shortcuts as described in
> https://bugs.kde.org/show_bug.cgi?id=323960
> 
> 1. Copy current URL to Clipboard
>   Default Shortcut: Ctrl-Alt-C
> 
> 2. Set current Url to the one in Clipboard
>   Default Shortcut: Ctrl-Alt-D (Because Ctrl-Alt-V is occupied by Klipper, D could be understood as "dispatch" maybe..
> 
> 
> Diffs
> -----
> 
>   dolphin/src/dolphinmainwindow.h 6b046c2 
>   dolphin/src/dolphinmainwindow.cpp a4dbb6f 
> 
> Diff: https://git.reviewboard.kde.org/r/115357/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Yichao Zhou
> 
>

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