<table><tr><td style="">filipf 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/D25584">View Revision</a></tr></table><br /><div><div><p>With this patch:</p>
<p>Fusion:<br />
<a href="https://phabricator.kde.org/F7788319" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F7788319: image.png</a></p>
<p>Windows 98:<br />
<a href="https://phabricator.kde.org/F7788322" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F7788322: image.png</a></p>
<p>Oxygen:<br />
<a href="https://phabricator.kde.org/F7788324" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F7788324: image.png</a></p>
<p>QtCurve:<br />
<a href="https://phabricator.kde.org/F7788326" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F7788326: image.png</a></p>
<p>Kvantum:<br />
<a href="https://phabricator.kde.org/F7788329" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F7788329: image.png</a></p>
<p>When the frame is enabled it just adds a double border with all widget styles so looks like an improvement to me. The screenshots do show we'll need to theme frame's border the same way we did for the tab bar's though.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R858 Qt Quick Controls 2: Desktop Style</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25584">https://phabricator.kde.org/D25584</a></div></div><br /><div><strong>To: </strong>filipf, Plasma, VDG, ngraham, broulik<br /><strong>Cc: </strong>broulik, ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>