D26758: Fix the size and pixel alignment of checkboxes and radiobuttons

[George Vogiatzis]

gvgeo added a comment.


  Plasma theme explorer is fine. The difference is that it uses plasmaComponents, while D26271 <https://phabricator.kde.org/D26271> uses plasmaComponents**3**.
  
  About plasmaComponents:
  There is some bug with the checkbox and the Label. Fails to take the correct size (uses paintedHeight).
  
    height: Math.round(Math.max(paintedHeight, theme.mSize(theme.defaultFont).height*1.6))
  
  Radiobutton draws correctly the label height. But need indicator vertical placement or different label height.
  
  ~~~~
  
  Non relevant note: 
  There is a workaround in multiple places for 
  https://bugreports.qt.io/browse/QTBUG-67007
  Which was fixed in Qt 5.14 from
  https://bugreports.qt.io/browse/QTBUG-70481
  Should be removed or need to stay awhile?

REPOSITORY
  R242 Plasma Framework (Library)

REVISION DETAIL
  https://phabricator.kde.org/D26758

To: ndavis, #plasma, #vdg, ngraham
Cc: gvgeo, ngraham, davidedmundson, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200128/a2d24c5d/attachment.html>