<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/D25728">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/D25728#572350" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D25728#572350</a>, <a href="https://phabricator.kde.org/p/manueljlin/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@manueljlin</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>But that style of line / divider was going to be after the toolbars too, like kirigami apps, so removing it just would make that inconsistent later on. The divider needs to be inside the application itself to make this possible, instead of in the titlebar.</p>

<p>If it's possible, fixing the color scheme and making it configurable should probably be the way to go.</p></div>
</blockquote>

<p>It's just wrong to locally draw a titlebar separator if there's a global option for it.</p>

<p>For consistency with the proposals see: <a href="https://phabricator.kde.org/D25728#571938" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D25728#571938</a></p>

<p>I can also say that after seeing Sierra Breeze Enhanced implementing most of what was decided there (only!) when the titlebar separator option is on that it's something that we should also do.</p>

<p><a href="https://phabricator.kde.org/F7799465" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7799465: sbe</a></p>

<p>KWin's titlebar option essentially becomes a "tools area distinguisher": turns on the titlebar separator (which gets hidden when app sends a hint that it has a tool area that is will separate on its own) and the different tool area color as well. Everything is consistent and in accordance to user choice.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</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>To: </strong>filipf, Plasma, VDG<br /><strong>Cc: </strong>manueljlin, anthonyfieroni, 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>