Review Request: Cleanup KLed

Christoph Feck christoph at maxiom.de
Fri Sep 17 13:03:52 BST 2010


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

(Updated 2010-09-17 12:03:51.864345)


Review request for kdelibs.


Changes
-------

Fix regression: Do not render square when shape is Rectangular.

Thanks Pedro for noticing.


Summary
-------

* massive code reduction by using Qt gradients
* fix background, now uses transparency
* fix cache not updated on look changes
* fix cropped appearance
* simplify cachedPixmap handling
* use SmallIconSize as sizeHint
* remove "builtin" benchmark

I would like to get feedback on the changed rendering of rectangular LEDs.


This addresses bug 98719.
    https://bugs.kde.org/show_bug.cgi?id=98719


Diffs (updated)
-----

  /trunk/KDE/kdelibs/kdeui/widgets/kled.h 1176221 
  /trunk/KDE/kdelibs/kdeui/widgets/kled.cpp 1176221 

Diff: http://svn.reviewboard.kde.org/r/5369/diff


Testing
-------


Screenshots (updated)
-----------

OLD appearance
  http://svn.reviewboard.kde.org/r/5369/s/505/
r1 appearance
  http://svn.reviewboard.kde.org/r/5369/s/506/
r3 appearance
  http://svn.reviewboard.kde.org/r/5369/s/508/


Thanks,

Christoph

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20100917/740adbca/attachment.htm>


More information about the kde-core-devel mailing list