<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/D10438" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D10438#204869" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D10438#204869</a>, <a href="https://phabricator.kde.org/p/januz/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@januz</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><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></p></div>
</blockquote>

<p>This is a touch based ui.</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>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></blockquote>

<p>None of these are application's menu. (though I did not check if applications menus are narrower in these three cases)</p>

<p>In anycase, increasing margins should then be made consistently accross the style and not one by one if you want to keep balance.<br />
For gnome, for instance, see how _all_ margins are larger (and thus consistent).</p>

<p>So far, on bugzilla there have been more complains about breeze being too space-hungry than too dense. <br />
(I can post the links here if needed)</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>