Using F2 and F9 in several cases, including reassigning those keys.
Details
- Reviewers
nmel martinkostolny yurchor - Group Reviewers
Krusader - Commits
- R167:53a5e29ccc44: Changes in several files because, by default, F2 renames and F9 opens a…
Diff Detail
- Repository
- R167 Krusader
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Thanks Toni. This patch fixes Edit and Tools menu on clean setups.
Unfortunately, it has no effect if user opened the app at least once as ~/.local/share/kxmlgui5/krusader/krusaderui.rc is never updated later.
including reassigning those keys.
I have these keys reassigned for a long time. It works fine regardless of this patch. The modified shortcuts are displayed in the menu correctly.
(independent of the patch) IMO, what a bad name scheme is picked for the actions. Names shouldn't contain default values. A better naming could be FN_Rename if one would like to denote these belong to the functional keys bar.
(independent of the patch) IMO, what a bad name scheme is picked for the actions. Names shouldn't contain default values. A better naming could be` FN_Rename` if one would like to denote these belong to the functional keys bar.
Yes, I think the same.
Additionally, I have made some related changes, including one in a docbook file.
The change in docbook is not relevant anymore.
https://cgit.kde.org/krusader.git/commit/?id=1d9988330eac92f1701f11131ec24f95c8d711eb
Nice catch, Toni. Please see my inline comment.
Please update the commit message accordingly. Thanks!
krusader/total_commander.keymap | ||
---|---|---|
3 | 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.