D26718: autostart KRunner with Plasma, aggregate text for KRunner in DesktopView

Piotr Dabrowski noreply at phabricator.kde.org
Sun Jan 19 20:56:51 GMT 2020


pdabrowski added a comment.


  In D26718#595756 <https://phabricator.kde.org/D26718#595756>, @davidedmundson wrote:
  
  > I don't want us to make krunner autostart. It's a hack, not a fix.
  
  
  I'm gonna play devil's advocate here :)
  If having an already running KRunner service is a hack, then why not terminate it each time after its job is done - and restart on request each time it is needed again?
  Bug #416145 says exactly this: "When krunner is invoked again during the same session it will work correctly."
  So you oppose preloading of KRunner service, but accept it continuing to run forever awaiting its next use. I don't find this consistent.
  
  IMHO (correct me if I'm wrong!):
  Something that is supposed to act immediately should be preloaded (sure: it may be a lighter version).
  Currently to show the KRunner window for the first time:
  
  - a call is made to D-Bus
  - which detects there is no KRunner service running
  - so it executes the KRunner binary
  - which then builds its UI
  - shows itself
  - and only then accepts user input
  
  This can't work fast. Meanwhile all user's keystrokes go to previously active window.

REPOSITORY
  R120 Plasma Workspace

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

To: pdabrowski, #plasma, #plasma_workspaces, davidedmundson, ngraham, broulik
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200119/d873d55a/attachment.html>


More information about the Plasma-devel mailing list