Review Request: Fixing painting of systemtray
Manuel Mommertz
2kmm at gmx.de
Tue Jul 13 21:07:59 CEST 2010
> On 2010-07-13 18:55:38, Marco Martin wrote:
> > /trunk/KDE/kdebase/workspace/plasma/generic/applets/systemtray/ui/applet.cpp, line 361
> > <http://reviewboard.kde.org/r/4623/diff/2/?file=31166#file31166line361>
> >
> > Qt::IntersectClip is needed in the case the systray would be in a clipped area eg in an item with the flag itemclipschildrentorect (e.g. the newspaper containment, or whatever containment that will have this charateristic)
> > otherwise the systray backgrond would bleed outside the clipped area instead of being cut
Didn't know this usecase. Changed to restore painter before applying clip
- Manuel
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4623/#review6538
-----------------------------------------------------------
On 2010-07-13 18:44:45, Manuel Mommertz wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/4623/
> -----------------------------------------------------------
>
> (Updated 2010-07-13 18:44:45)
>
>
> Review request for Plasma.
>
>
> Summary
> -------
>
> Correct drawing code in systemtray.
> don't use rect() anymore as this rect doesn't contain borders in planar more and leads to drawing errors there. setting to same size as contentsRect.
> Right easment is currently not used and therefore removed from calculation.
> Qt::IntersectClip doesn't make sense here as this clips away the background for firstelements.
>
> For 4.6 I would completly rewrite the drawing code for better reading. But to not introduce new bugs in 4.5 in this late phase I want to get this patch for now.
>
>
> Diffs
> -----
>
> /trunk/KDE/kdebase/workspace/plasma/generic/applets/systemtray/ui/applet.cpp 1148491
>
> Diff: http://reviewboard.kde.org/r/4623/diff
>
>
> Testing
> -------
>
>
> Screenshots
> -----------
>
> systray in floating layout. bottom with patch applied
> http://reviewboard.kde.org/r/4623/s/458/
>
>
> Thanks,
>
> Manuel
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20100713/a2f17123/attachment.htm
More information about the Plasma-devel
mailing list