<table><tr><td style="">ngraham edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-qkreekels32gk53/" 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;">This implements the menu shadow changes proposed in D9549; now we use a combobox with <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">four</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">five</span> options<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> (</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">: </span>None, Small,<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> Medium, Large, and Very Large.<br />
<br />
Large is the new default value (64px window shadows, 16px menu/tooltip shadows), and implements what we recently changed the default to. Small is the old default value (16px window shadows,</span> <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);">12px menu/tooltip shadows) for people who preferred that.<br />
<br />
Details:<br />
<br />
None (1px border for windows</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">and Large) to choose the window shadow size.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">menus,</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Normal is what we recently changed the default to (and this remains the new default) and Small is the old default value for people who preferred that.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">and tooltips):<br />
{F5623033}<br />
<br />
Small (16px window shadows,</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">None gives the window a single-pixel border instead of a shadow,</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">12px menu/tooltip shadows);</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">and Large gives a 96px</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">replicates the the old default value:<br />
{F5623034}<br />
<br />
Medium (32px window</span> shadow<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">s,</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Menu and </span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">14px menu/</span>tooltip shadows<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> are now 25% the size of the main</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">):<br />
{F5623035}<br />
<br />
Large (64px</span> window shadows<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> for Large and Normal</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">37% for Small,</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">16px menu/tooltip shadows);</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">and 0% for None (they get the same gets a single-pixel border).</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">the new default:<br />
{F5623036}</span><br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Very Large (96px window shadows, 24px menu/tooltip shadows):<br />
{F5623038}<br />
<br />
<br />
</span>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>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>