<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/D8232" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Hi,<br />
Thanks for the patch.<br />
Few comments (on the functionality, not yet on the code)</p>

<p>1/ this is a decision VDG should make (Thomas Pfeiffer ?): when I first wrote the code for breeze we discussed this possibility to have the shadow centered (as well as left oriented) but it was rejected. The right oriented shadow being a concious choice of the main breeze designer at the time, Andrew Lake</p>

<p>2/ if the option is added it must be applied consistently everywhere. This includes (as already noticed): menu and combobox shadows, but also buttons shadows. This requires changes to breeze::style as well as breeze::decoration.<br />
The shadow code for menu and comboboxes is 'similar' to that of the decorations. The button shadow is somewhat different, but ...</p>

<p>3/ I don't think this should be an option. The change is too small (and implies too much code complexity, once 2/ is fullifille) for being controled by an option, and doesn't bring/remove anything functionality wise, instead of eye candy. This would be like adding an option on how many pixels are used to round corners at the top of the window decoration. Bottom line one should choose one design or the other, it is not worth it to keep both. On the other hand we must then be prepared to people complaining when the change is introduced, and have a good story for justifying the change (rather than just : because I think it looks better).</p>

<p>4/ note that oxygen has centered shadows everywhere (decoration, menus and buttons).</p>

<p>Comments welcome</p>

<p>Hugo</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/D8232" rel="noreferrer">https://phabricator.kde.org/D8232</a></div></div><br /><div><strong>To: </strong>rpelorosso, Breeze, VDG, hpereiradacosta<br /><strong>Cc: </strong>rkflx, ngraham, davidedmundson, mart, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>