Bug 328944 - Use Shift+Delete for delete or move selected files to trash
Summary: Use Shift+Delete for delete or move selected files to trash
Status: RESOLVED FIXED
Alias: None
Product: krusader
Classification: Applications
Component: general (show other bugs)
Version: 2.4.0-beta3 "Single Step"
Platform: Mint (Ubuntu based) Linux
: NOR normal
Target Milestone: ---
Assignee: Krusader Bugs Distribution List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-12-18 00:08 UTC by Cade
Modified: 2018-05-06 00:15 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Cade 2013-12-18 00:08:36 UTC
If "Settings" -> "Configure Krusader" -> "General" -> "General" -> "Delete mode" -> "Delete files"
is active
1. key "Delete" must delete selected files,
2. "Shift+Delete" must move selected files to trash.

If "Settings" -> "Configure Krusader" -> "General" -> "General" -> "Delete mode" -> "Move to trash"
is active
1. key "Delete" must move selected files to trash,
2. "Shift+Delete" must delete selected files.

Or
1. allow user bind key "delete";
2. add action "Move to trash" to dialog "Settings" -> "Configure shortcuts";
3. allow user bind "Shift+Delete".

Reproducible: Always

Actual Results:  
user can't bind "Shift+Delete" to any action.
"Shift+Delete" may be used only for "Cut".
Comment 1 Alex Bikadorov 2017-03-01 19:38:01 UTC
Git commit dfc4abb00ffaa1508056e172a42c7d074b08870a by Alexander Bikadorov.
Committed on 01/03/2017 at 19:34.
Pushed by abikadorov into branch 'master'.

Panel: DEL+CTRL always inverts default delete mode (move to trash vs. real deletion)

M  +2    -2    krusader/Panel/krpopupmenu.cpp
M  +2    -2    krusader/Panel/krview.cpp
M  +2    -2    krusader/Panel/krview.h
M  +1    -1    krusader/Panel/listpanel.cpp
M  +1    -1    krusader/Panel/listpanelactions.cpp
M  +8    -4    krusader/Panel/panelfunc.cpp
M  +2    -1    krusader/Panel/panelfunc.h

https://commits.kde.org/krusader/dfc4abb00ffaa1508056e172a42c7d074b08870a
Comment 2 Alex Bikadorov 2017-03-09 19:36:07 UTC
Patch should be good enough. If not, please reopen.