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