D21037: Add deselect action

[Elvis Angelaccio]

elvisangelaccio added a comment.


  In D21037#464000 <https://phabricator.kde.org/D21037#464000>, @faridb wrote:
  
  > In D21037#463984 <https://phabricator.kde.org/D21037#463984>, @elvisangelaccio wrote:
  >
  > > 1. You can already deselect everything by pressing an empty space in the view, if you are a mouse user.
  >
  >
  > This is not always possible. For example, if you're using the //details view mode// and there is not empty space to click (when the scrollbar is active).
  
  
  Can you show a screenshot of a detail view affected by such problem?
  
  > In D21037#463984 <https://phabricator.kde.org/D21037#463984>, @elvisangelaccio wrote:
  > 
  >> 2. You can already deselect everything by pressing the `Esc` key, if you are a keyboard user.
  > 
  > 
  > I actually didn't know this.
  > 
  > What about adding the deselect action to the menu and disabling its default keyboard shortcut which conflicts with the shortcut for invert selection?
  >  The deselect action will be available in the menu without being assigned a keyboard shortcut which will be useful for mouse users. Keyboard users are already able to use `Esc`.
  
  Hmm, I don't know, normal actions should always have a shortcut.

REPOSITORY
  R318 Dolphin

REVISION DETAIL
  https://phabricator.kde.org/D21037

To: faridb, #dolphin, #vdg, elvisangelaccio
Cc: mmustac, ngraham, elvisangelaccio, kfm-devel, alexde, feverfew, meven, spoorun, navarromorales, firef, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20190512/614ec91d/attachment.htm>