----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126551/ -----------------------------------------------------------
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
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
