<table><tr><td style="">anemeth updated this revision to Diff 25350.<br />anemeth added a comment.<br />Restricted Application edited projects, added KWin; removed Plasma.
</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/D9848" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Reverted the change of putting the blur strength configuration into a vector, because it resulted in some complications.<br />
Doing it with the huge switch case might be unpleasent for the eyes but it does not require more unnecessary variables and<br />
(in my opinion) is cleaner and more maintainable this way.</p>

<p>I strongly disagree with the intent to remove these magic numbers and let the user only change the offset or the iteration number.<br />
There aren't enough strength steps between the steps if we make one of these values constant.<br />
If we do this we remove a big part of customization and KDE is all about customization.<br />
Those 15 set of magic number numbers is the only price we have to pay for the customization.</p>

<p>Said that, if the core members of Plasma or KWin decide/vote to make the offset constant and let the user only change<br />
between the 4 or 5 iteration number to make the code a bit leaner at the cost of customization then I will not be against the decision.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D9848?vs=25305&id=25350" rel="noreferrer">https://phabricator.kde.org/D9848?vs=25305&id=25350</a></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/D9848" rel="noreferrer">https://phabricator.kde.org/D9848</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>.arcconfig<br />
effects/blur/blur.cpp<br />
effects/blur/blur.h<br />
effects/blur/blur.kcfg<br />
effects/blur/blur_config.ui<br />
effects/blur/blurshader.cpp<br />
effects/blur/blurshader.h</div></div></div><br /><div><strong>To: </strong>anemeth, Plasma, KWin<br /><strong>Cc: </strong>broulik, romangg, zzag, anthonyfieroni, mart, davidedmundson, fredrik, ngraham, plasma-devel, kwin, KWin, iodelay, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol<br /></div>