D27438: Set System Tray Plasmoid spacing value from 0 to units.smallSpacing

Nathaniel Graham noreply at phabricator.kde.org
Sun Feb 16 18:51:28 GMT 2020


ngraham resigned from this revision.
ngraham added a comment.


  I have a vertical panel and use large tray icons but intentionally minimize their number, and the result looks fine to me IMO: F8107338: Screenshot_20200216_115030.png <https://phabricator.kde.org/F8107338>
  
  However I'll admit that I thought the status quo was fine too. So I have no particular opinion on the aesthetics of this patch, but I think we should be careful to avoid adding too much spacing because the people who have and like lots of tray icons have a legitimate concern here. To me, `units.smallSpacing` (i.e. 2) is the absolute max we should do.
  
  To make this patch easier to review without applying it, please add all the various sets of before-and-after images to the Test Plan section, not in comments.

REPOSITORY
  R120 Plasma Workspace

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

To: The-Feren-OS-Dev, #vdg
Cc: ndavis, anthonyfieroni, davidre, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200216/b4a368a1/attachment.html>


More information about the Plasma-devel mailing list