<table><tr><td style="">ngraham 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/D27845">View Revision</a></tr></table><br /><div><div><p>So now with this patch, the new default panel height is 46px. It seems like there's still a camp that would prefer a smaller value, though and I don't want to steamroll over them. However I agree with <span class="phabricator-remarkup-mention-unknown">@veggero</span> that the 42 and 44px sizes look odd given the current icon scaling algorithms, so I'd prefer not to use those. I also happen to think that 40 looks odd too, since the Kickoff icon is visibly larger than the icons for pinned apps: <a href="https://phabricator.kde.org/F8290199" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8290199: 40.png</a></p>

<p>The system tray items do seem to fit the space better with the thinner panel though. But I don't think that's a huge deal since tray icons are supposed to be small and unobtrusive.</p>

<p>Also from a less technical angle, I think that when using an IOTM, a larger effective click area is desirable since the buttons are now much less wide than TTM items are. This is especially true for touch use cases, which we need to consider since 2-in-1s are a thing. So personally, I think the extra size imparted by 46px is useful.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27845">https://phabricator.kde.org/D27845</a></div></div><br /><div><strong>To: </strong>ngraham, VDG, Plasma<br /><strong>Cc: </strong>mart, filipf, kmaterka, gvgeo, davidedmundson, ndavis, niccolove, The-Feren-OS-Dev, plasma-devel, Orage, LeGast00n, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra<br /></div>