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

List:       kde-panel-devel
Subject:    Re: [Panel-devel] [PATCH] Small "fixes"
From:       "Aaron J. Seigo" <aseigo () kde ! org>
Date:       2007-07-13 15:31:34
Message-ID: 200707130931.34451.aseigo () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Friday 13 July 2007, Rafael Fernández López wrote:
> Well as Aaron proposed on other mail, ExtenderItem is probably not needed,
> and maybe is worth using directly Plasma::Applet. It is very annoying that

hm.. why not just QGraphicsItem?

> What I have noticed is that we need this changes for making it being drawn
> correctly (for example when they are faked-drawn on a dragging event to
> another extender, that they are painted with 50% of opacity). This makes
> them being painted and being painted in the correct place.

ok.. looking at the code some more, imho the translate call should go into 
Extender::paint right before the d->itemBeingDraggedInto->paint line so that 
the painter is "pre-translated". perhaps right after the painter->setOpacity 
call?

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

Full time KDE developer sponsored by Trolltech (http://www.trolltech.com)

[Attachment #5 (application/pgp-signature)]

_______________________________________________
Panel-devel mailing list
Panel-devel@kde.org
https://mail.kde.org/mailman/listinfo/panel-devel


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