Review Request: config

[Chani]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/639/
-----------------------------------------------------------

(Updated 2009-05-03 12:29:46.686527)


Review request for Plasma.


Changes
-------

ok, last change before I commit :) squished a couple of regressions.

-made sure containments always have a config action (aaron, there's a FIXME line in setHasConfig... because the code doesn't look quite right)
-set the text for remove & configure at the right time so that everyone gets the text they expect. I'm not 100% happy with it because it sets the text in two places, but it's the best I could come up with.


Summary
-------

-needs testing without my local patch that fixes a qt shortcut bug


Diffs (updated)
-----

  /trunk/KDE/kdelibs/plasma/applet.h 961912 
  /trunk/KDE/kdelibs/plasma/applet.cpp 961912 
  /trunk/KDE/kdelibs/plasma/containment.cpp 961912 
  /trunk/KDE/kdelibs/plasma/corona.h 961912 
  /trunk/KDE/kdelibs/plasma/corona.cpp 961912 
  /trunk/KDE/kdelibs/plasma/private/applet_p.h 961912 
  /trunk/KDE/kdelibs/plasma/private/containment_p.h 961912 

Diff: http://reviewboard.kde.org/r/639/diff


Testing
-------


Thanks,

Chani