-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4623/
-----------------------------------------------------------

(Updated 2010-07-15 10:41:09.211652)


Review request for Plasma.


Changes
-------

readd rightEasement and save pointer again after restore (didn't know that 
restore is a pop-function)


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 (updated)
-----

  /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

Reply via email to