Review Request 126551: [Service Runner] WIP: Add support for Jump List Actions

[Kai Uwe Broulik]

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

(Updated Jan. 30, 2016, 7:32 nachm.)


Status
------

This change has been discarded.


Review request for Plasma and Vishesh Handa.


Repository: plasma-workspace


Description
-------

This adds support for Jump List Actions to the Services runner

I wanted to make the Jump Lists themselves searchable, so you can also enter "Big Picture" and it finds the associated service with it but I couldn't figure out how I can search Desktop Actions through KServiceTypeTrader. Commented in the code is a crude approach.

Also, I can't figure out how I could stick those entries toegether, it should always be "Main App - action - action - action - Some other App", sometimes it shuffles them again although I tried hard to have them stick together.

I need a better way to distinguish app results and jump list results, I used the match type but that obviously doesn't work now. I don't want QString matching the data, though.

Finally, I want those results not to show up in Kickoff, is this possible?

I removed the context.isValid() check in the loops as it's done once before all of them.


Diffs
-----

  runners/services/servicerunner.h a4171bc 
  runners/services/servicerunner.cpp 27df5ab 

Diff: https://git.reviewboard.kde.org/r/126551/diff/


Testing
-------

Searched for "steam", got Steam as well as its associated actions
Searched for "spectacle", got its associated capture modes


File Attachments
----------------

Steam in KRunner
  https://git.reviewboard.kde.org/media/uploaded/files/2015/12/28/9f3046f1-6acb-460d-a515-6abab6522dfe__krunnerjumplists.png


Thanks,

Kai Uwe Broulik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160130/5d42cc0d/attachment.html>