[Panel-devel] minimization to taskbar

[Aaron J. Seigo]

On Saturday 01 December 2007, Jason Stubbs wrote:
> On Sunday 02 December 2007 00:00:52 Robert Knight wrote:
> This was the "works" patch. ;)
>
> I am aware that paint() isn't a good place to do it though. Even listening
> for ItemPositionChanged isn't enough as that event only happens when the
> position has changed with regard to the immediate parent. I guess I will
> need to add an event listener like I did with the system tray. I wonder if
> that functionality couldn't be moved into Widget as well...

i like the idea of this going into Widget when it is ready. i don't think 4.4 
will remove the need for that code as it's mapping between the sceen and x11 
geometries, something i'm not sure is there even in the 4.4 widget support? 
(i could be wrong on that)

also, Robert mentioned cached painting in 4.4 .. we already have that in 
libplasma and it's pretty much identical to what is in 4.4, so if it works 
now it should work in qt 4.4 the same way from a painting perspective.

> > - Both TaskGroupItem and AbstractTaskItem implement
> > publishIconGeometry(), which means that when both the task and the
> > parent (a TaskGroupItem) are repainted, both will publish icon
> > geometry for the task.
>
> I had guessed that this would happen, but wasn't really going to worry
> about it. Is it a problem? Even after getting position and size change
> handling above correct, animations will cause the icon geometry publish to
> occur several times a second. If it's not a problem, I'd like to leave this

these can easily be compressed with a timer, though. also, the layout animator 
is rather broken right now and needs to be fixed so that items stop animating 
a *lot* sooner than they do right now (run with QT_FLUSH_PAINT to see the 
full effect of the animations); the animator is doing too many small fine 
adjustments, particularly at the end, that just don't matter visually.

-- 
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/20071201/43874881/attachment.pgp