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

Reply via email to