----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4623/#review6575 -----------------------------------------------------------
Ship it! much better now - Marco On 2010-07-15 10:41:09, Manuel Mommertz wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/4623/ > ----------------------------------------------------------- > > (Updated 2010-07-15 10:41:09) > > > 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 > 1149766 > > 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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
