<div dir="ltr"><div>I'll do what the VDG decides.</div><div>but this is fixing an actuall issue so maybe it's worth to accept and then fine tune later?</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 21, 2019 at 8:45 PM Noah Davis <<a href="mailto:noreply@phabricator.kde.org">noreply@phabricator.kde.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><table><tbody><tr><td>ndavis added a comment.
</td><td><a style="text-decoration:none;padding:4px 8px;margin:0px 8px 8px;float:right;color:rgb(70,76,92);font-weight:bold;border-radius:3px;background-color:rgb(247,247,249);background-image:linear-gradient(rgb(255,255,255),rgb(241,240,241));display:inline-block;border:1px solid rgba(71,87,120,0.2)" href="https://phabricator.kde.org/D21301" target="_blank">View Revision</a></td></tr></tbody></table><br><div><div><blockquote style="border-left:3px solid rgb(140,152,184);color:rgb(107,116,140);font-style:italic;margin:4px 0px 12px;padding:8px 12px;background-color:rgb(248,249,252)">
<div style="font-style:normal;padding-bottom:4px">In <a href="https://phabricator.kde.org/D21301#467973" style="background-color:rgb(231,231,231);border-color:rgb(231,231,231);border-radius:3px;padding:0px 4px;font-weight:bold;color:black;text-decoration:none" target="_blank">D21301#467973</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="color:rgb(25,85,141);background-color:rgb(241,247,255);border:1px solid transparent;border-radius:3px;font-weight:bold;padding:0px 4px" target="_blank">@ngraham</a> wrote:</div>
<div style="margin:0px;padding:0px;border:0px none;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 rgb(167,181,191);color:rgb(70,76,92);font-style:italic;margin:4px 0px 12px;padding:4px 12px;background-color:rgb(248,249,252)"><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:rgb(231,231,231);border-color:rgb(231,231,231);border-radius:3px;padding:0px 4px;font-weight:bold;color:black;text-decoration:none" target="_blank">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:rgb(231,231,231);border-color:rgb(231,231,231);border-radius:3px;padding:0px 4px;font-weight:bold;color:black;text-decoration:none" target="_blank">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:rgb(231,231,231);border-color:rgb(231,231,231);border-radius:3px;padding:0px 4px;font-weight:bold;color:black;text-decoration:none" target="_blank">#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="gmail-m_7979296014199081807remarkup-link" rel="noreferrer" target="_blank">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:rgb(235,235,235) none repeat scroll 0% 0%;font-size:13px">view-fullscreen</tt> icon is a maximize button, wouldn't it be better to use <tt style="background:rgb(235,235,235) none repeat scroll 0% 0%;font-size:13px">window-maximize</tt> or <tt style="background:rgb(235,235,235) none repeat scroll 0% 0%;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" target="_blank">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></blockquote></div>