<table><tr><td style="">cameronrodgers 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/D23480">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/D23480#521324" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D23480#521324</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>Actually I found a UI regression that happens with this patch. There's now a white/gray line underneath the title:</p>

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

<p>Good catch. This seems to be related to the Blur effect, as it doesn't occur if Blur is turned off in Desktop Effects (in my KDE Neon VM). I agree it's a regression for the title bars, but this problem is actually more widespread. The current code (without this patch applied) already has the white/gray/transparent line around icons, and around (some) title bars. Here is a screenshot of the current Present Windows effect:<br />
<a href="https://phabricator.kde.org/F7289990" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7289990: grayline.png</a><br />
Do we have any clues on how to prevent this? I'll be looking into it later.</p>

<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/D23480#521330" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D23480#521330</a>, <a href="https://phabricator.kde.org/p/davidedmundson/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@davidedmundson</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>If we do remove the highlight (and I'm willing to say, that's purely aesthetics and therefore not my realm) we should remove all the code that does the highlight.</p>

<p>This still animates, and still updates every frame during the animation only to show the same value.</p></div>
</blockquote>

<p>Unfortunately, it appears that the highlight variable is also used in window scaling calculations:<br />
<a href="https://github.com/KDE/kwin/blob/master/effects/presentwindows/presentwindows.cpp#L369" class="remarkup-link" target="_blank" rel="noreferrer">https://github.com/KDE/kwin/blob/master/effects/presentwindows/presentwindows.cpp#L369</a><br />
As a result, I think most or all of the code calculating the "highlight" still plays some role. Since "highlight" would no longer actually have anything to do with window brightness, would it be appropriate to rename the variables and functions to reflect the change in functionality?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23480">https://phabricator.kde.org/D23480</a></div></div><br /><div><strong>To: </strong>cameronrodgers, KWin, VDG, Plasma, ngraham<br /><strong>Cc: </strong>davidedmundson, ngraham, zzag, romangg, kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart<br /></div>