<table><tr><td style="">januz 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/D10438" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>IMO, the last version looks better than the current menu. That said, I think the top/bottom paddings are still too tight, I would try adding 2-3px for each.<br />
It's true that there's a question of taste but more whitespace is generally a good thing (unless you go overboard and start making huge widgets). A couple more pixels in the menus will help focus the elements better (by framing them in negative space), it will make the ui look less "full of stuff" and less tense.</p>

<p>For reference:</p>

<p>Material design manual: <a href="https://material.io/guidelines/components/menus.html#menus-usage" class="remarkup-link" target="_blank" rel="noreferrer">https://material.io/guidelines/components/menus.html#menus-usage</a><br />
Gnome: <a href="http://i.imgur.com/er2odvE.png" class="remarkup-link" target="_blank" rel="noreferrer">http://i.imgur.com/er2odvE.png</a><br />
Mac: <a href="https://www.intego.com/mac-security-blog/wp-content/uploads/2016/12/Mac-menu-bar-extras-sound.png" class="remarkup-link" target="_blank" rel="noreferrer">https://www.intego.com/mac-security-blog/wp-content/uploads/2016/12/Mac-menu-bar-extras-sound.png</a><br />
Windows: <a href="https://docs.microsoft.com/en-us/windows/uwp/design/controls-and-patterns/menus" class="remarkup-link" target="_blank" rel="noreferrer">https://docs.microsoft.com/en-us/windows/uwp/design/controls-and-patterns/menus</a></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/D10438" rel="noreferrer">https://phabricator.kde.org/D10438</a></div></div><br /><div><strong>To: </strong>zzag, Breeze, VDG, ngraham, hpereiradacosta<br /><strong>Cc: </strong>januz, fabianr, mmustac, abetts, anemeth, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart<br /></div>