<table><tr><td style="">ngraham edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-eoi7cioaacvglhs/" rel="noreferrer">(Show Details)</a><br />ngraham edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-xj2n7kfxawckyzj/" rel="noreferrer">(Show Details)</a>
</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/D9627" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>{F56230<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">33}</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">69}</span><br />
<br />
Small (16px window shadows, 12px menu/tooltip shadows); replicates the the old default value:<br />
{F56230<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">34}</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">71}</span><br />
<br />
Medium (32px window shadows, 14px menu/tooltip shadows):<br />
{F56230<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">35}</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">72}</span><br />
<br />
Large (64px window shadows, 16px menu/tooltip shadows); the new default:<br />
{F56230<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">36}</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">73}</span><br />
<br />
Very Large (96px window shadows, 24px menu/tooltip shadows):<br />
{F56230<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">38}</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">74}</span><br />
<br />
<br />
I had a massive amount of help from @hpereiradacosta; in fact, probably 75% of these changes are his. It's a shame multiple authorship isn't possible. Hugo, feel free to commandeer the revision if you'd like the credit!</div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>Upgrade story: since this changes the way shadow size is stored in the breezerc file, users who previously had manually set their shadow size to some arbitrary pixel value will now get the new default <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Normal</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Large</span> 64 px shadow size.</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R31 Breeze</div></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/D9627" rel="noreferrer">https://phabricator.kde.org/D9627</a></div></div><br /><div><strong>To: </strong>ngraham, VDG, Breeze, hpereiradacosta, abetts<br /><strong>Cc: </strong>rkflx, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>