nmel accepted this revision.
nmel added a comment.

  In D12524#254117 <https://phabricator.kde.org/D12524#254117>, @asensi wrote:
  
  > In D12524#254087 <https://phabricator.kde.org/D12524#254087>, @nmel wrote:
  >
  > > Nice catch, Toni. [...]
  > >  [Do I recall correctly that Totcmd doesn't have any of these shortcuts: 
Rename, Terminal, Multirename? I don't remember exactly because I probably had 
defined them a long time ago as I use F2 and F9 without hassle. :) ]
  >
  >
  > Total Commander has those internal commands and those shortcuts by default:
  >
  >   cm_ExecuteDOS       [None that I have found]
  >   cm_RenameOnly       Shift+F6
  >   cm_MultiRenameFiles Ctrl+M
  >
  > and you can assign a shortcut to an internal command, I suppose that that's 
what was meant :-)
  
  
  I confirmed today that Shift+F6 and Ctrl+M work in Total Commander as F2 and 
Shift+F2 in Krusader. I see Ctrl+M is assigned for media action in keymap... :/
  
  I think it's fine to merge it as is because the patch is clearly an 
improvement and the keymap needs a more fundamental update.

REPOSITORY
  R167 Krusader

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

To: asensi, #krusader, nmel, martinkostolny, yurchor
Cc: kde-doc-english, yurchor, martinkostolny, nmel, mjanczara, miroslavm, 
janlepper, abika, asensi, gengisdave, andreaska

Reply via email to