D27438: Increase System Tray Plasmoid spacing value slightly

[Nathaniel Graham]

ngraham added a comment.


  https://community.kde.org/Infrastructure/Phabricator#Marking_patches_as_dependent_on_other_patches
  
  It would probably be simplest do just do this though:
  
  1. Abandon this patch
  2. Submit a patch to refactor that logic to use a nested multi-line function for readability, as proposed in my comment
  3. After that patch lands, submit two more patches, one to increase the spacing when in tablet mode, and another one to increase the spacing when in desktop mode. At this point each patch will be changing a different line in the function
  
  Clear as mud? :)

REPOSITORY
  R120 Plasma Workspace

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

To: The-Feren-OS-Dev, #vdg, ngraham, davidre
Cc: broulik, kmaterka, filipf, 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/20200217/407e38a2/attachment.html>