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
