Review Request 119920: Restore original checkbox and radio button size

Kai Uwe Broulik kde at privat.broulik.de
Sat Sep 6 07:46:01 UTC 2014


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

(Updated Sept. 6, 2014, 7:46 vorm.)


Review request for Plasma.


Changes
-------

gridUnit is mSize


Repository: plasma-framework


Description
-------

This patch restores the original checkbox and radio button size calculation as it was before the migration to QtQuick Controls.

On one hand the newer, smaller one size-wise looks closer to the one used in Widget/Desktop-style apps but eg. battery monitor looks weird with the smaller one.


Diffs (updated)
-----

  src/declarativeimports/plasmacomponents/qml/styles/CheckBoxStyle.qml d9dc9fd 
  src/declarativeimports/plasmacomponents/qml/styles/RadioButtonStyle.qml 6d06f01 

Diff: https://git.reviewboard.kde.org/r/119920/diff/


Testing
-------

CheckBox looks like it did before, did not test Radio Button, in Radio Button it also does away with the hardcoded 6px margin used previously.


Thanks,

Kai Uwe Broulik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140906/71cbeb8f/attachment-0001.html>


More information about the Plasma-devel mailing list