Review Request: config

Chani chanika at
Sun May 3 21:29:46 CEST 2009

This is an automatically generated e-mail. To reply, visit:

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

Review request for Plasma.


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.


-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 





More information about the Plasma-devel mailing list