API review: protected config of applet

Aaron J. Seigo aseigo at kde.org
Thu Apr 24 17:16:33 CEST 2008


On Thursday 24 April 2008, Marco Martin wrote:
> Hi all,
>
> i was trying to get congig() globalConfig() etc protected, but i
> encountered some problems.
> there areKickoff and desktop containment that have things that access the
> applet config from outside the applet.
> for Kickoff the problem is the Launcher class, the QWidget that manages all
> the menu things, so should still store his config into applet config?
> making it a friend of applet? or making the applet co copy a pointer to his
> config into launcher with something like Launcher::setConfig()?

KickOff could declare config() to be public. but yes, i think this is a 
pattern that is likely to be repeated, so perhaps we should mark this item 
for re-examination and leave it public for now.

> or even taking it as an excuse to kill IconLoader? (probably should be done
> one day or another)

yes, i think we should do this and commit to delivering a working folder view 
in 4.1.

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Trolltech
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 194 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20080424/320c8aa7/attachment.pgp 


More information about the Panel-devel mailing list