<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/D9549" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Please no commit. The patch is _incomplete_.<br />
As already stated, there is a need for a matching change in the widget style, for the same shadow to apply to menus, tooltips, and combobox dropdown. <br />
I have such a change ready locally already. Just have not had time to submit updated review, and asked for comments.<br />
Please be patient. (or make this patch complete).<br />
Also, I have the feeling, testing the local patch i had in real life for a while, that the top side of the shadow at least, should be less strong than the other three sides.<br />
Comments welcome.</p>

<p><a href="https://phabricator.kde.org/p/abetts/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@abetts</a>: good direction is not enough. The decoration is very exposed to users. We do not want a half  backed product to go into production code ( = release), with many iterations changes, etc. This just has bad impact on the quality of the style.</p>

<p>By lack of time I have had no time to implement the other suggestion done by ALake (who is also VDG + the original designer of the widget style and decoration), so that I agree that this (once complete) should be committed, with an option, when ready for commit (which is not the case now).</p>

<p>Best,</p>

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