<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/D25334">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/D25334#563346" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D25334#563346</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>Found the bug report tracking that requested change: <a href="https://bugs.kde.org/show_bug.cgi?id=370465" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=370465</a></p>

<p>Based on the comments there, it's likely that this proposal would receive pushback, and we should endeavor to understand why it was changed to the current state in the first place. That's not a reason to hold back on doing it, we just need to understand the history and make the change for legitimate usability-based reasons so it's not perceived as "flipping things back and forth over and over again".</p></div>
</blockquote>

<p>The relevant quote:</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>Kai Uwe Broulik 2018-04-03 07:51:10 UTC <br />
 We had exactly what you suggest for a while and then it was changed back to line art. Can we stop randomly changing theme whenever someone comes by and doesn't like something unless it actually comes enclosed with a 20 page empirical study made in a lab or something?</p></blockquote>

<p>Looks like he's referring to using only colored backgrounds, which is not what we're doing now. I do think the current way of showing window state is visually a bit confusing.</p>

<p>Here's what I think:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">The line should indicate that the app is opened, the window is not minimized and the window is focused (opaque blue) or inactive (opaque gray)</li>
<li class="remarkup-list-item">The background should indicate whether an app is not opened (no BG), minimized or inactive (semi-transparent gray), or focused (semi-transparent blue)</li>
</ul>

<p>This way it mostly doesn't change, but the visuals have more of a logical relation to each other.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25334">https://phabricator.kde.org/D25334</a></div></div><br /><div><strong>To: </strong>niccolove, VDG, ngraham<br /><strong>Cc: </strong>ngraham, ndavis, KonqiDragon, VDG, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns<br /></div>