From kfm-devel Tue Jan 28 16:03:51 2014 From: "Frank Reininghaus" Date: Tue, 28 Jan 2014 16:03:51 +0000 To: kfm-devel Subject: Re: Review Request 115357: Add a shortcut for copying current directory Message-Id: <20140128160351.11809.69485 () probe ! kde ! org> X-MARC-Message: https://marc.info/?l=kfm-devel&m=139092505315791 MIME-Version: 1 Content-Type: multipart/mixed; boundary="--===============3144352680589178025==" --===============3144352680589178025== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit > 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 > > --===============3144352680589178025== Content-Type: text/html; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit
This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115357/

On January 28th, 2014, 3:58 p.m. UTC, 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


On January 28th, 2014, 3:55 p.m. UTC, Yichao Zhou wrote:

Review request for Dolphin.
By Yichao Zhou.

Updated Jan. 28, 2014, 3:55 p.m.

Bugs: 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)

View Diff
--===============3144352680589178025==--