<table><tr><td style="">hpereiradacosta 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/D11069">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Some questions:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Do we really need different shadow sizes for menus, etc? Can we use single shadow size in kstyle?</li>
</ul></blockquote>

<p>I'd say yes, as soon as you provide different shadow sizes for the windows (that also answers the last question)</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<li class="remarkup-list-item">Do menus have outlines? If you look at picture above, you can see grey "gap" between menu and submenu.</li>
</ul></blockquote>

<p>yes menus have outlines. In fact, in the old shadows, windows also had an outline. (drawn with the shadow).<br />
We want to keep it as far as I know.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<li class="remarkup-list-item">What's the best way to share code between KStyle and KDecoration plugin? (currently, I'm symlinking files)</li>
</ul></blockquote>

<p>File links is not an option.  (it breaks when you make tarballs of when compiling on e.g. windows)<br />
You would need to move the files away in a separate folder, compile them in a library, link the library to both kstyle and kdecoration. Oxygen does that.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>In addition to the first question, it's a little bit weird that changing window decoration shadow size also changes shadows "in apps" (e.g. menu shadows would have different size, etc).</p></blockquote>

<p>Why weird ? if I want "large shadows" I want them large for everything that have shaodws (with possible hierarchy). And if I want small I want them all small. <br />
That seems natural to me.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R31 Breeze</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11069">https://phabricator.kde.org/D11069</a></div></div><br /><div><strong>To: </strong>zzag, Breeze, VDG, hpereiradacosta<br /><strong>Cc: </strong>fabianr, hpereiradacosta, ngraham, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>