<table><tr><td style="">filipf created this revision.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />filipf 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/D25728">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>I added a top separator for QML applet configuration windows in <a href="https://phabricator.kde.org/D20908" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;">D20908</a></p>

<p>This separator is the equivalent of KWin's separator, minus the highlight color.</p>

<p>I have been regretting the decision to draw it ever since the patch was comitted and believe it to be wrong for several reasons:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">it's an infrigment on user choice: it doesn't respect the fact the user has turned off KWin's titlebar separator</li>
<li class="remarkup-list-item">it's inconsistent with qwidgets: there is no top separator there</li>
<li class="remarkup-list-item">it's degrades the goal of trying to have a uniform appearance between the window manager and window in the case of Breeze Light and Breeze Dark because it distinctly points out the titlbar</li>
<li class="remarkup-list-item">it's superfluous (not visible) when used with the default color scheme anyway</li>
</ul>

<p>With this patch only the sidebar is enclosed, which solves all of the points above with the exception of the second one (which just warrants this patch be replicated for qwidgets)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>BRANCH</strong><div><div>no-unwanted-titlebar-separator (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25728">https://phabricator.kde.org/D25728</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>desktoppackage/contents/configuration/AppletConfiguration.qml</div></div></div><br /><div><strong>To: </strong>filipf<br /><strong>Cc: </strong>plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>