[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: Extender api review, round 2
From:       "Aaron J. Seigo" <aseigo () kde ! org>
Date:       2008-07-29 16:38:55
Message-ID: 200807291038.55630.aseigo () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Tuesday 29 July 2008, Rob Scheepmaker wrote:
> I've had a lot of feedback on the extender api, and now I've incorporated a
> lot of the suggestions into a more up to date api. Again, I'm looking
> forward to getting feedback to fine tune this api all the way.

it's looking better and better! =)

in Extender i found myself thinking "maybe it would feel more natural if 
instead of Extender::extenderItems it was just Extender::items..." and then i 
got to the signals which are just itemAdded and itemRemoved =) so perhaps 
change all the "extenderItem"s to "item"s in Extender's API.

ExtenderItem::setExpirationTimeWhenAttached could be 
ExtenderItem::setAutoExpireDelay? setAutoExpireTime? 

ExtenderItem::collapsed should be isCollapsed

i like how the impact on Applet is now so minimal, clean and obvious! =) 

as for multiple extenders per-Applet, let's just stick with one for now i 
think. we can learn how they get used and move on from there.

one final note, adding a new virtual to Applet makes libplasma BIC... hm.. i'll 
start a new thread about that issue actually...

> Oh and another question, does anybody know if there's any way to determine
> if a top level QWidget is in front of another top level QWidget? Because to
> make drag & drop reliable, there really should be a way to determine which
> plasma view is on top at a specific location. And QApplication::topLevelAt
> won't do, since while dragging, there might be a graphicsView under the
> mouse cursor when dragging from view to view.

KWindowSystem::stackingOrder gives you the current window stacking order.

-- 
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


["signature.asc" (application/pgp-signature)]

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[prev in list] [next in list] [prev in thread] [next in thread]