ngraham added a comment.

  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 @veggero 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: F8290199: 40.png 
<https://phabricator.kde.org/F8290199>
  
  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.
  
  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.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D27845

To: ngraham, #vdg, #plasma
Cc: 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

Reply via email to