Review Request: Enable css border rendering on styled select elements

Andrea Iacovitti aiacovitti at libero.it
Fri May 18 10:35:57 BST 2012


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

Review request for kdelibs.


Description
-------

The patch enable rendering of css border for styled select elements (combobox+listview).
This is done by using KdeUiProxyStyle as already done by other widgets in khtml.
For styled combobox the drop down arrow is rendered by drawing PE_IndicatorArrowDown of the base style,
so it will look different depending on the system style being in use.


This addresses bugs 299934 and 300227.
    http://bugs.kde.org/show_bug.cgi?id=299934
    http://bugs.kde.org/show_bug.cgi?id=300227


Diffs
-----

  khtml/css/html4.css 9f4d17b 
  khtml/rendering/render_form.h 2be4df5 
  khtml/rendering/render_form.cpp b4cf136 

Diff: http://git.reviewboard.kde.org/r/104983/diff/


Testing
-------

. khtml testregression -> no further test fails
. tested against different basestyle: oxygen, qtcurve, available qt styles (plastique, qcleanlooks ...)
. tested against Qt 4.7.4, Qt 4.8.1
. daily browsing


Screenshots
-----------

shoot of a bko page before/after the patch
  http://git.reviewboard.kde.org/r/104983/s/573/


Thanks,

Andrea Iacovitti

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120518/4e55d26c/attachment.htm>


More information about the kde-core-devel mailing list