Holding shift while the right-click menu is displayed turns the 'Move to Trash' action into a 'Delete' action. This is all working correctly, and it is a feature that I really appreciate.
The issue only happens when the 'Delete' action is added to the context menu through the configuration options (Settings->Configure Dolphin...->Services then check 'Delete'). Then, the 'Move to Trash' option does not change visually in the right-click menu when shift is held, but clicking 'Move to Trash' while shift is held deletes the file rather than moving it to the trash.
As a solution, I see two options:
1. Even when the delete action is in the right click menu, change the 'Move to Trash' item to a 'Delete' item when shift is held. This would result in two 'Delete' actions being shown if shift is held.
2. When there is already a 'Delete' item in the right-click menu, simply remove the 'Move to Trash' item when shift is held.
Steps to Reproduce:
1. Add the 'Delete' item to the right click menu (Settings->Configure Dolphin...->Services then check 'Delete')
2. Right click on a file (or select several files and then right click)
3. Hold shift button down
The 'Move to Trash' icon does not change visually, but the behavior changes so that clicking on the 'Move to Trash' icon deletes the item instead of moving it to the trash.
Either the 'Move to Trash' item should change to a 'Delete' item when shift is held, or the 'Move to Trash' item should be removed when shift is held.
Note that without the 'Delete' item added to the menu through the configuration, the 'Move to Trash' item correctly changes into a 'Delete' item when shift is held.
I'm using KDE 4.9.1 in pre-release Kubuntu 12.10 (quantal) on an ASUS Eee 1215b.
Thanks for the bug report, I can confirm the problem. DolphinContextMenu does not update the text if both actions are shown (which is good), but unfortunately, also DolphinViewActionHandler::slotTrashActivated() tries to be clever and deletes the item(s) if shift is pressed.
Suggested fix at https://git.reviewboard.kde.org/r/107509/.
Git commit fda03a10e725ba133a32a032f7f456d839e0d800 by Frank Reininghaus.
Committed on 09/06/2013 at 11:53.
Pushed by freininghaus into branch 'master'.
Do not delete files when Shift-clicking "Trash"
Thanks to Dawit Alemayehu for making this fix possible with commit
M +1 -8 dolphin/src/views/dolphinviewactionhandler.cpp