<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/D20908">View Revision</a></tr></table><br /><div><div><p>I'm so in love with this new style. it's just astonishingly good-looking.</p>

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

<p>Once this is in users' hands (in conjunction with no borders by default for Breeze), I predict that we get a flood of praise for it.</p>

<p>Implementation-wise I wonder if this is the correct place to locate the horizontal line below the titlebar. In this case it works, but we have other windows that currently need it too (e.g. System Settings, Discover, probably any Kirigami app). I wonder if it should be drawn by a framework instead, or the Breeze style, or even KWin. Or some combination of the two.</p>

<p>For more on the subject of horizontal lines under titlebars, see the discussion in <a href="https://phabricator.kde.org/T10201#183425" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T10201#183425</a>.</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/D20908">https://phabricator.kde.org/D20908</a></div></div><br /><div><strong>To: </strong>filipf, VDG, Plasma, ngraham<br /><strong>Cc: </strong>broulik, GB_2, hein, plasma-devel, jraleigh, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>