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