<table><tr><td style="">ndavis 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><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/D21301#467973" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D21301#467973</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Fantastic. The dark inactive tab is maybe a bit too dark, but that's a fairly minor thing.</p></div>
</blockquote>

<p>I agree, the header for the prompt on the right seems too dark because text has poor contrast.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><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></blockquote>

<p>Without separation, it reminds me of old Safari: <a href="https://www.uberphones.com/wp-content/uploads/2011/06/ipad-safari-tabs-update.jpg" class="remarkup-link" target="_blank" rel="noreferrer">https://www.uberphones.com/wp-content/uploads/2011/06/ipad-safari-tabs-update.jpg</a></p>

<p>I don't think it's a big deal, but it might be nice to have a separating line.</p>

<p>Also, if the button with the <tt style="background: #ebebeb; font-size: 13px;">view-fullscreen</tt> icon is a maximize button, wouldn't it be better to use <tt style="background: #ebebeb; font-size: 13px;">window-maximize</tt> or <tt style="background: #ebebeb; font-size: 13px;">window-restore</tt>?</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>ndavis, shubham, konsole-devel, gennad, thsurrel, ngraham, maximilianocuria, hindenburg<br /></div>