Review Request: just show an icon as configoverlay for popupapplets in panel

Marco Martin notmart at gmail.com
Sat Nov 22 22:58:01 CET 2008


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.vidsolbach.de/r/277/
-----------------------------------------------------------

Review request for Plasma.


Summary
-------

some applets like twitter have an overlay and a config button over them.
but if a popupapplet has this configuration required the config button is in the panel, so it's too large for the icon and exits from the layout.
this patch makes it a plasma icon with the usual config icon in that case, not sure about putting a so specialized case but looks way better


Diffs
-----

  /trunk/KDE/kdelibs/plasma/applet.cpp

Diff: http://reviewboard.vidsolbach.de/r/277/diff


Testing
-------


Thanks,

Marco



More information about the Plasma-devel mailing list