<table><tr><td style="">rkflx 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/D10530" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>+1</p>
<p>FWIW, I support this revert. I think introducing this was more of a misunderstanding in relation to being used to a particular look, because by default in Windows (and elsewhere) there are almost no icons in menus, so you are used to this look:</p>
<p><a href="https://phabricator.kde.org/F5712769" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;" rel="noreferrer">F5712769: win-menu.png</a></p>
<p>In KDE's software icons are much more prevalent, so you have less empty space. However, note that in the Windows screenshot (and on other platforms too) icons are in place of the empty space, while your patch added extra empty space next to the icons (i.e. more extreme than Windows).</p>
<p>In my book, adding extra space only when required for checkboxes (like we did before the patch) is just fine.</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/D10530" rel="noreferrer">https://phabricator.kde.org/D10530</a></div></div><br /><div><strong>To: </strong>zzag, Breeze, hpereiradacosta<br /><strong>Cc: </strong>rkflx, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>