<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/D27122">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/D27122#608443" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D27122#608443</a>, <a href="https://phabricator.kde.org/p/niccolove/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@niccolove</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Using Background for panels would break the visual integration with dialogs, so I would avoid doing that.</p>

<p>Regarding panel popups / dialogs; I'm already working on the toparea patch - <a href="https://phabricator.kde.org/D27189" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D27189</a> - but I was thinking, the problem with using Background is the controls on top of it e.g. kickoff search. Since the transparency makes the background color not as gray, the only problem is when things are opaque (I remember we talked about this in the VDG chat). Now, *there is* a version of Breeze opaque in the breeze/opaque/ directory which is used when compositing is disabled. For that one, I did leave ColorScheme-Background, so the problem is not there when the actual opaque version is used. I think that people who want things to be 100% opaque will install a theme rather than manually editing the svg, so they would not have that problem as well. That isn't to say that I won't work on the top area, just that I would not make the this patch dependent on it.</p>

<p>I also tried to tweak the intensity value to make the normal ColorScheme-Background fine, but I did not have any success. It seems like it's not a color which manages to work in this usecase, while ColorScheme-ViewBackground is okay.</p></div>
</blockquote>

<p>This is really about how this fits with our color scheme system, not so much about the aesthetics of transparency and blur. If you need different colors from normal to get this to look right, how will it look when you use other different colors? How does this look with color schemes besides Breeze and Breeze Dark (e.g., Arc Dark, Materia Dark)? If it only works properly with Breeze/Breeze Dark because ViewBackground is a lighter/darker color, this isn't going to work for all color schemes. You'll end up having to use a custom colors file anyway (as in Breeze Light/Dark desktop themes) to guarantee that everything will look right.</p>

<p>Now that I've had the chance to try this patch, I can some issues. Even if you use a custom colors file, it seems this patch only looks right when there are no windows open, which doesn't seem realistic.</p>

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

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

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

<p>Even Breeze Dark has trouble: <a href="https://phabricator.kde.org/F8095608" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8095608: Screenshot_20200210_114056.PNG</a></p>

<p>That's not to say the current situation is perfect, but the color difference isn't so stark (Breeze Dark): <a href="https://phabricator.kde.org/F8095617" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8095617: Screenshot_20200210_114353.PNG</a></p>

<p>Light themes don't really have this problem, but dark themes have a big problem.</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/D27122">https://phabricator.kde.org/D27122</a></div></div><br /><div><strong>To: </strong>niccolove, ndavis, ngraham, cblack<br /><strong>Cc: </strong>cblack, ndavis, filipf, ngraham, davidedmundson, kde-frameworks-devel, manueljlin, Orage, LeGast00n, The-Feren-OS-Dev, konkinartem, ian, jguidon, Ghost6, jraleigh, zachus, MrPepe, fbampaloukas, squeakypancakes, alexde, IohannesPetros, GB_2, trickyricky26, ragreen, michaelh, crozbo, ZrenBot, firef, bruns, alexeymin, skadinna, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, aaronhoneycutt, abetts, sebas, apol, ahiemstra, mbohlender, mart<br /></div>