<table><tr><td style="">ndavis added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D26758">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D26758#601718" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;">D26758#601718</a>, <a href="https://phabricator.kde.org/p/gvgeo/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@gvgeo</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Plasma theme explorer is fine. The difference is that it uses plasmaComponents, while <a href="https://phabricator.kde.org/D26271" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;">D26271</a> uses plasmaComponents<strong>3</strong>.</p>

<p>About plasmaComponents:<br />
 There is some bug with the checkbox and the Label height. Fails to take the correct size (uses paintedHeight).</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">height: Math.round(Math.max(paintedHeight, theme.mSize(theme.defaultFont).height*1.6))</pre></div>

<p>Radiobutton draws correctly the label height. But need indicator vertical placement or different label height.</p></div>
</blockquote>

<p>Sorry, I don't understand what the problem is and I don't know the QML code well. I didn't touch that code (I think), so I'm not sure it has anything to do with this patch. Can you state the problem in simpler terms?</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Non relevant note: <br />
 There is a workaround in multiple places for <br />
 <a href="https://bugreports.qt.io/browse/QTBUG-67007" class="remarkup-link" target="_blank" rel="noreferrer">https://bugreports.qt.io/browse/QTBUG-67007</a><br />
 Which was fixed in Qt 5.14.0 RC1 from<br />
 <a href="https://bugreports.qt.io/browse/QTBUG-70481" class="remarkup-link" target="_blank" rel="noreferrer">https://bugreports.qt.io/browse/QTBUG-70481</a><br />
 Should be removed or need to stay awhile?</p></blockquote>

<p>I'd suggest removing them when the minimum Qt requirement is raised up to or past that version. If the workarounds cause visual glitches, then ask whoever maintains the affected repos about it.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26758">https://phabricator.kde.org/D26758</a></div></div><br /><div><strong>To: </strong>ndavis, Plasma, VDG, ngraham<br /><strong>Cc: </strong>gvgeo, ngraham, davidedmundson, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns<br /></div>