[Panel-devel] [PATCH] Beginnings of a panel implementation for discussion

[Aaron J. Seigo]

On Friday 17 August 2007, Robert Knight wrote:
> This is mainly here to discuss the API for the Panel class and so that
> we can put something together for the applet developers to test with.

there are many issues with this code as it stands, but i'm fine with using it 
as a start. issues, in no particular order, none of which imho block you 
putting this into svn as a draft right now:

- the dptr in Panel needs to be const'd

- can you put it under the "LGPLv2 or later" please? thanks....

- KWindowSystem::setOnAllDesktops(winId(), true);  is of course correct and 
not questionable at all =)

- style guide is not followed consistently (some day soon someone is going to 
poison my food or something for being such a grump about that ;)

- why another layout instead of just using HBoxLayout for horizontal panels 
and VBoxLayout for verticals? i suppose this is a property of using its own 
Corona at the moment.

- the really interesting thing is how to manage multiple panels on screen. i 
suppose that will probably be left to me to figure out based on my experience 
with kicker (it's a remarkably non-trivial problem =) though i'm happy to be 
pleasantly surprised.

- it does need to share the main corona, as matt noted, but that can wait. 
this is blocked by the background painters, which may sound odd at first but 
it isn't when one considers they are the natural containers for applets (and 
where form factors will also be moving =). this is something i'm already 
working on, so don't worry about it too much right now. when this is in svn, 
we can also address the hardcoded applets then

- we need to integrate some composite manager bling here

- size, position setting/loading also needs to be set up, but i have ideas 
there as well and this can wait for a week or two..

thanks for getting this ball rolling.

-- 
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: 189 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20070817/120f9d3b/attachment.pgp