D6056: [Run Command] Auto-close on focus loss only if query field is empty

[Kai Uwe Broulik]

broulik created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  If the user didn't accept any search result or didn't explicitly close it, he or she might still want to tweak the query, e.g. when summing multiple numbers but first having to look them up.
  
  BUG: 353026
  FIXED-IN: 5.11.0

TEST PLAN
  In theory, pressing Alt+Space twice (one to give KRunner focus and another time to close it again) should close it but on X this doesn't work as the global shortcut grabs and then it loses focus in the instant it tries to determin whether it should close or not (Wayland fixed it :P)
  
  Getting this in early in the cycle would give it a lot of testing to see whether it becomes too annoying.

REPOSITORY
  R120 Plasma Workspace

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

AFFECTED FILES
  krunner/view.cpp
  lookandfeel/contents/runcommand/RunCommand.qml

To: broulik, #plasma, #vdg
Cc: plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170601/d968f647/attachment.html>