Review Request 126514: [Milou/KRunner] Support runner actions
Kai Uwe Broulik
kde at privat.broulik.de
Mon Dec 28 15:49:45 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126514/
-----------------------------------------------------------
(Updated Dez. 28, 2015, 3:49 nachm.)
Review request for Plasma, KDE Usability and Vishesh Handa.
Changes
-------
* Implement keyboard navigation
Using Tab you can now reach the actions as well, ie, you tab this way:
"David" - "Start chat" - "Open log viewer" - (next entry) "Another guy" - "Start chat" - "Open log viewer" - (next result and so on
vice-versa when back-tabbing. Return invokes the highlighted action. I had to abuse the checked state for this as we cannot programmatically pretend to be hovered. Arrow keys move between search results as usual.
Repository: milou
Description
-------
This brings back support for additional runner actions. A runner can specify supplemental actions to a search result.
It is deliberately no ToolButton as that would look weird, the Highlight blends in much better, although I'm not too happy with the huge Plasma tooltip. I'm not sure how keyboard interaction should work as both Tab and arrow keys would conflict with navigation.
Diffs (updated)
-----
lib/CMakeLists.txt 1123b60
lib/qml/ResultDelegate.qml 6d60bf2
lib/qml/ResultsView.qml b36cc19
lib/sourcesmodel.h 6c2d205
lib/sourcesmodel.cpp 65c69b3
Diff: https://git.reviewboard.kde.org/r/126514/diff/
Testing
-------
Hardly any runner makes use of this, though.
I got the kill runner to send sigterm and sigkill, and also some mysterious ktp runer I couldn't find where it is offers "Start Chat" and "Open Chat Log" actions. The Solid runner supposedly also adds actions but I couldn't get that to work.
Unfortunately I had to add QtWidgets for QAction :(
File Attachments
----------------
Runner action
https://git.reviewboard.kde.org/media/uploaded/files/2015/12/25/3b70cc42-f3f9-42e5-aa5c-3b30067ae06e__krunneractions.png
Thanks,
Kai Uwe Broulik
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151228/90e06d3d/attachment.html>
More information about the Plasma-devel
mailing list