<table><tr><td style="">sitter created this revision.<br />sitter added a reviewer: sandsmark.<br />Restricted Application added a subscriber: kde-utils-devel.
</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/D8941" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>QGB with some styles has very unfortunate rendering if it has no title<br />
nor frame configured. e.g. with the windows 9x style it renders the top<br />
border line assuming there is a title even when there is none.<br />
Ultimately we do not need QGB here at all though as we have it entirely<br />
hidden and only use it as glorified layout container. So, replace QGB with<br />
a Layout directly. Reduces excess code, memory footprint and fixes<br />
appearance with various styles.</p>

<p>(technically we could put the entire setup into the designer file and<br />
 do away with the QButtonGroup as radios are autoexclusive with other<br />
 radios in the same parenting qwidget anyway; seems unncessarily invasive<br />
 though)</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>no more silly line artifact with windows 9x style; breeze continues to look stelar</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R352 Filelight</div></div></div><br /><div><strong>BRANCH</strong><div><div>no-color-groupbox</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8941" rel="noreferrer">https://phabricator.kde.org/D8941</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/dialog.ui<br />
src/settingsDialog.cpp</div></div></div><br /><div><strong>To: </strong>sitter, sandsmark<br /><strong>Cc: </strong>kde-utils-devel<br /></div>