<table><tr><td style="">ngraham created this revision.<br />ngraham added reviewers: KWin, VDG.<br />Herald added a project: KWin.<br />Herald added a subscriber: kwin.<br />ngraham requested review of this revision.
</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/D22886">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The QML Window Decorations KCM uses a fake tab bar with code copied from Plasma-pa, due<br />
to the lack of a real QML tab widget (see <a href="https://bugs.kde.org/show_bug.cgi?id=394296" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=394296</a>).<br />
As a result, we have to manufacture our own by combining a TabBar with a frame in which<br />
the content sits, and any change made to one needs to be applied to the other as well.</p>
<p>The Plasma-pa version just received a minor change in <a href="https://phabricator.kde.org/D22533" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: line-through;">D22533</a>. This patch implements the<br />
same change for the Window Decorations KCM so they don't look inconsistent with one<br />
another.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Before-and-afters are identical to the images in <a href="https://phabricator.kde.org/D22533" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: line-through;">D22533</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>BRANCH</strong><div><div>slightly-improve-fake-tabbar (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22886">https://phabricator.kde.org/D22886</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcmkwin/kwindecoration/package/contents/ui/main.qml</div></div></div><br /><div><strong>To: </strong>ngraham, KWin, VDG<br /><strong>Cc: </strong>kwin, LeGast00n, fmonteiro, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, mart<br /></div>