Review Request: fix rendering of klineedit clear button

Hugo Pereira Da Costa hugo at oxygen-icons.org
Sun Apr 25 03:30:36 BST 2010


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

Review request for kdelibs.


Summary
-------

this patch
- fixes the rendering of animated klineedit clear button, to use QPainter compositing mode rather than QPainter::setOpacity to fake semi-transparent clearButton during animation. This should be more efficient as QPainter::setOpacity (last time I checked) was using the rasterEngine. 

- uses a QIcon rather than QPixmap to store the clearButton pixmap. This allows to have the proper icon effect when rendering enabled/disabled klineedit. 


Diffs
-----

  /trunk/KDE/kdelibs/kdeui/widgets/klineedit_p.h 1118236 

Diff: http://reviewboard.kde.org/r/3804/diff


Testing
-------

in konqueror and dolphin, as well as custom made test widgets.
Test done in klineedit, as well as editable kcombobox
Using either oxygen, cleanlooks and plastik style. 

Finally, tested with/without the animation being enabled


Thanks,

Hugo





More information about the kde-core-devel mailing list