Extender api review round 4

[Aaron J. Seigo]

On Tuesday 02 September 2008, Rob Scheepmaker wrote:
> - allow using qgraphicsitems in ExtenderItems instead of just QGWs. I agree
> with this suggestion, just haven't implemented it yet. I will soon.

excellent.

> - No virtual initExtenderItem in Applet. I really doubt this is a good
> idea. 

yes, leave it there imho.

> - No extender() in Applet. That is certainly a possibility. This point is
> mainly an aesthetic one. Personally, I think having the extender() function
> leads to nicer code.

agreed.

> - Theming. Currently the dragger svg, contains a preferred-icon-size-hint
> element. This decides which size the icons in the draghandle will get, and
> also decides which vertical size the draghandle gets (iconsize + margins).
> I can certainly see different themes having different emphasis on saving
> screen real estate vs having nice, big, easy to hit with you mouse icons,
> and this makes it possible. It does however feel a bit hackish, and was
> wondering if there's a nicer way to accomplish this.

no, it's fine as is. just be sure to document it thoroughly at 

	http://techbase.kde.org/Projects/Plasma/Theme

hm.. perhaps we should move that content to kdebase/worskpace/plasma/design

> - Detached extender items. When an ExtenderItem gets detached, the
> ExtenderApplet applet (in playground atm) get's loaded, and the
> ExtenderItem is added to it's extender (since ExtenderItems are always
> contained in an extender). Not only does this bring the issue of a part of
> libplasma being dependent on an applet, it also feels wrong to have a
> certain applet hardcoded as the default extender container. Possible
> solution for that last problem would maybe be something like associating
> applets with a plasma/extenderitem mime type or something like that, but
> still, I think it should work without requiring the user to have a certain
> applet installed. Any suggestions?

put it in libplasma in the private/ folder.

-- 
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: 197 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20080910/d4dc4807/attachment-0001.sig