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