<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/D21301">View Revision</a></tr></table><br /><div><div><p>Fantastic. The dark inactive tab is maybe a bit too dark, but that's a fairly minor thing.</p>

<p>One remaining UI niggle I can see is that I'd like a horizontal line separating the headerbars from the menubar for the case where the tab bar is hidden. When the tab bar is shown, it provides its own line, which looks good: <a href="https://phabricator.kde.org/F6841233" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6841233: Screenshot_20190521_094624.png</a></p>

<p>But when the tab bar is hidden, there's no separation and it looks a bit weird: <a href="https://phabricator.kde.org/F6841236" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6841236: Screenshot_20190521_094723.png</a></p>

<p>Or maybe this isn't a big deal? What do you think? <a href="https://phabricator.kde.org/tag/vdg/" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">#VDG</a>?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21301">https://phabricator.kde.org/D21301</a></div></div><br /><div><strong>To: </strong>tcanabrava, Konsole, hindenburg, ngraham, VDG<br /><strong>Cc: </strong>konsole-devel, gennad, thsurrel, ngraham, maximilianocuria, hindenburg<br /></div>