<table><tr><td style="">ngraham 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#572362" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D25728#572362</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>I though that would just affect every app's titlebar without actually checking if the app has a toolbar, etc etc and just add/remove the divider everywhere and be done with it. However, if the app can actually send a hint to Kwin to make it hide or show the divider depending if it has a toolbar or not (or maybe through a window / app rule), then it's great</p></div>
</blockquote>

<p>Exactly.</p>

<p>We can't rely 100% on KWin to draw the separator because with our proposed redesign, we want to have the separator below the menu or toolbar, if those are present. That means they would need to drawn by the app or the widget toolkit, not the window manager. That means that we can't have KWin always draw or not draw the titlebar. That means we need to have the separator drawn here.</p>

<p>However there are two caveats that I can accept:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Right now this is the only kind of window that gains a separator drawn by the content view itself, so we should either turn it off or add it to all other windows with only titlebars</li>
<li class="remarkup-list-item">We should make the content view not draw the titlebar when KWin is drawing it</li>
</ul>

<p>Maybe what we should do is change the titlebar separator color in Breeze to be a pleasant dark gray like in the screenshot in <span class="phabricator-remarkup-mention-unknown">@flipwise</span>'s latest comment instead of a garish blue, turn it on by default, and remove this here. Then for Plasma 6 (or whenever we do the proposed Breeze evolution) we can revisit that implementation and come up with something smarter.</p>

<p>Does that make sense?</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>