Review Request: Add action support to krunner default interface
Jacopo De Simoi
wilderkde at gmail.com
Sat Nov 21 19:21:00 CET 2009
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/2259/
-----------------------------------------------------------
Review request for Plasma and Aaron Seigo.
Summary
-------
This patch adds action support to the default interface by adding a button for each one of actionsForMatch given by the runnerManager.
Hovering (and focusing) over a button changes the subtext of the item according to the selected action.
The logic is all there, it has still some rough edges from the graphical side (e.g. we should make sure that there are not "too many" displayed actions) and focusing with the keyboard is missing (it should be done togheter with the configuration button) but these can be fixed quite easily once it gets committed.
Diffs
-----
trunk/KDE/kdebase/workspace/krunner/interfaces/default/interface.h 1052428
trunk/KDE/kdebase/workspace/krunner/interfaces/default/interface.cpp 1052428
trunk/KDE/kdebase/workspace/krunner/interfaces/default/resultitem.h 1052428
trunk/KDE/kdebase/workspace/krunner/interfaces/default/resultitem.cpp 1052428
trunk/KDE/kdebase/workspace/krunner/interfaces/default/resultscene.h 1052428
trunk/KDE/kdebase/workspace/krunner/interfaces/default/resultscene.cpp 1052428
Diff: http://reviewboard.kde.org/r/2259/diff
Testing
-------
the only runner supporting actions I am aware of is the Devices (solid) runner (in kdereview); tested with that and it works.
Screenshots
-----------
Actions for the device runner
http://reviewboard.kde.org/r/2259/s/269/
Thanks,
Jacopo
More information about the Plasma-devel
mailing list