D10961: Add default shortcut "/" for opening filter panel

[Roman Inflianskas]

rominf added a comment.


  In D10961#217070 <https://phabricator.kde.org/D10961#217070>, @ngraham wrote:
  
  > Works as advertised, and doesn't conflict with any default shortcuts. The whole "type / to quick search/filter" is pretty well established  in web browsers nowadays. Makes sense to have it in Dolphin, too.
  >
  > Konsole still eats it when its panel is open, but it also eats the other [ctrl] + [I] shortcut even without this patch, so that seems like another bug (and down the rabbit hole we go...)
  >
  > It might be nice to take the opportunity to modernize the syntax <https://doc.qt.io/qt-5.9/qkeysequence.html#details> for the existing shortcut here, replacing the `|` with a `+` for readability, since you're touching that code anyway.
  
  
  Done
  
  > But now, to completely confuse you and undermine my credibility on the matter( ;-) ), let me recommend against including unrelated `#include` changes in this patch. There's a fine line to walk here, and personally, I think it's okay to modernize a piece of code that you happen to be touching, but it's best not to include unrelated code modernizations elsewhere in the file. Cleaning up those redundant `#includes` is a good idea, but should be separately, and preferably all at once, so it's easy to see if it causes any unexpected regressions.
  
  OK. Reverted those changes. Do you mind if I do massive commit (for the whole project) that cleans up the code?

REPOSITORY
  R318 Dolphin

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

To: rominf, #dolphin, ngraham
Cc: ngraham, #dolphin, spoorun, navarromorales, isidorov, firef, andrebarros, emmanuelp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180303/a0c9d4fa/attachment.htm>