Review Request 129251: Remove Shift+Del as secondary shortcut for Cut

[Albert Astals Cid]

> On Oct. 30, 2016, 10:27 a.m., David Faure wrote:
> > Right. We have Shift+Del as an alternate shortcut for cut because when I switched from Windows to Linux (last millenium!), I had taken the habit to use Ctrl+Insert, Shift+Del, and Shift+Insert for copy/cut/paste (these are certainly more logical than Ctrl+C/X/V...).
> > 
> > Indeed this shortcut doesn't work for file managers, who have to get rid of it. But the majority of apps are not file managers, so the current situation seems fine to me (file managers remove the shortcut, everyone else gets it). => I vote for discard.
> 
> Mark Gaiser wrote:
>     What would be your shortcut of choice for permanently deleting files?
>     Meta + Delete?
>     or just nothing and force the user to clear the trash to permanently delete files?
> 
> David Faure wrote:
>     IMHO permanent deletion is Shift+Delete, in filemanagers -- the two alternatives you offer seem extremely bad to me ;)
>     
>     Permanent deletion of emails is Shift+Delete too (and there cut isn't supported), so that's consistent.
>     
>     Even while I was using Shift+Delete as my preferred shortcut for cut (bad laptop keyboards forced me to switch away from that), it never bothered me that it would delete and not cut in filemanagers. Maybe because I never used cut very much in filemanagers, I admit.
>     I suppose this forces one to learn Ctrl+X for cutting in dolphin.
>     
>     All this being said, I can very well imagine that not many people use Ctrl+Insert, Shift+Del, and Shift+Insert... I don't have a strong case if I'm not using that myself anymore. It just seems harmless to support this in apps where it doesn't conflict with anything else, and to give priority to other shortcuts in apps where they do conflict (i.e. permanent deletion with an easy shortcut has priority over the less-commonly-used-alternate-shortcut-for-cut).

My suggestion at https://git.reviewboard.kde.org/r/129299/


- Albert


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129251/#review100392
-----------------------------------------------------------


On Oct. 24, 2016, 10:47 a.m., Elvis Angelaccio wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129251/
> -----------------------------------------------------------
> 
> (Updated Oct. 24, 2016, 10:47 a.m.)
> 
> 
> Review request for KDE Frameworks, KDE Usability and Matthew Dawson.
> 
> 
> Bugs: 357747
>     https://bugs.kde.org/show_bug.cgi?id=357747
> 
> 
> Repository: kconfig
> 
> 
> Description
> -------
> 
> This patch removes Shift+Del as secondary shortcut for the Cut action. This shortcut was set back in 2001.
> 
> Reasons for removing it:
> 
> * The expected standard behavior for this shortcut is "Permanently delete"
> * For the reason above, it is also set as primary shortcut for the `DeleteFile` action. This causes conflicts in applications.
> * For the reason above, many applications (e.g. Dolphin or Digikam) already resolve this conflict on their own.
> 
> Credits to Jan for the investigation: https://bugs.kde.org/show_bug.cgi?id=347373#c2
> 
> 
> Diffs
> -----
> 
>   src/gui/kstandardshortcut.cpp 92eb091382c7ab2110240cef21f29268be787250 
> 
> Diff: https://git.reviewboard.kde.org/r/129251/diff/
> 
> 
> Testing
> -------
> 
> Using Shift+Del in Gwenview now works as expected.
> 
> 
> Thanks,
> 
> Elvis Angelaccio
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20161031/ccbedae8/attachment-0001.html>