<table><tr><td style="">cameronrodgers created this revision.<br />cameronrodgers added reviewers: KWin, VDG, Plasma.<br />cameronrodgers added a project: KWin.<br />Herald added a subscriber: kwin.<br />cameronrodgers requested review of this revision.
</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><strong>REVISION SUMMARY</strong><div><p>This patch removes the dimming on windows in Present Windows, but keeps the dimming on the background. It also moves the title bars to the bottom of the windows to further increase readability and ease of visual recognition. In order to prevent title bars overlapping windows, the spacing between windows in flexible grid is increased a bit. Here is a screenshot:<br />
<a href="https://imgur.com/a/ZUKDsD1" class="remarkup-link" target="_blank" rel="noreferrer">https://imgur.com/a/ZUKDsD1</a><br />
IMO it is clearly much better looking and more useful this way.</p>

<p>There is another revision being discussed that only changes window dimming, and includes a setting for it in the configs:<br />
<a href="https://phabricator.kde.org/D8388" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D8388</a><br />
I think people in that discussion may be overthinking it. Shouldn't we make the simplest change first (before adding outline effects, etc)? I haven't seen anyone indicate that they actually like the darkening effect. It's visually confusing. Why not just get rid of it outright?</p>

<p>Please feel free to merge this thread/mark duplicate if you think it's appropriate (though this isn't an exact duplicate, and changes the title bars), I am new and don't know the methodology here. I just hope that these two issues are fixed soon, since they are simple fixes.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Apply patch to kwin, compile, and run to see the change.</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>AFFECTED FILES</strong><div><div>effects/presentwindows/presentwindows.cpp</div></div></div><br /><div><strong>To: </strong>cameronrodgers, KWin, VDG, Plasma<br /><strong>Cc: </strong>kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, mart<br /></div>