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

[David Faure]

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



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.

- David Faure


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/20161030/74fc5193/attachment.html>