<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/D27438">View Revision</a></tr></table><br /><div><div><p><a href="https://community.kde.org/Infrastructure/Phabricator#Marking_patches_as_dependent_on_other_patches" class="remarkup-link" target="_blank" rel="noreferrer">https://community.kde.org/Infrastructure/Phabricator#Marking_patches_as_dependent_on_other_patches</a></p>

<p>It would probably be simplest do just do this though:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">Abandon this patch</li>
<li class="remarkup-list-item">Submit a patch to refactor that logic to use a nested multi-line function for readability, as proposed in my comment</li>
<li class="remarkup-list-item">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</li>
</ol>

<p>Clear as mud? :)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27438">https://phabricator.kde.org/D27438</a></div></div><br /><div><strong>To: </strong>The-Feren-OS-Dev, VDG, ngraham, davidre<br /><strong>Cc: </strong>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<br /></div>