<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham added a comment.<br />This revision now requires changes to proceed.
</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/D28353">View Revision</a></tr></table><br /><div><div><p>I see some problems here.</p>

<p>To be honest, the level of transparency is way too high. I know aesthetics are subjective, but it just doesn't look good to me. Even with our default wallpaper, something about it feels off:</p>

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

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

<p>In addition, on the usability side, every transparent surface has worse contrast than it did before. And while your screenshots look appealing enough when there are no windows on the screen or the wallpaper is otherwise visible. But that's not really how people use their machines, right? people tend to maximize their windows (novice users or those with small screens) or tile everything (advanced users) or have a dozen or more un-maximized, un-tiled windows in a huge jumble taking up every pixel of the screen (messy users).</p>

<p>The effect produced when a very transparent popup appears over window content of the type that any of these users will be looking at just looks really bad to me; the background becomes an ugly gray:</p>

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

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

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

<p>Also the pop-up still doesn't match the panel background: <a href="https://phabricator.kde.org/F8201861" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8201861: Screenshot_20200327_112208.png</a>. Is there another patch I'm missing that implement this?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28353">https://phabricator.kde.org/D28353</a></div></div><br /><div><strong>To: </strong>niccolove, VDG, Plasma, cblack, ngraham<br /><strong>Cc: </strong>ngraham, cblack, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns<br /></div>