<table><tr><td style="">konstantinshtepa 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/D4204" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D4204#80142" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D4204#80142</a>, <a href="https://phabricator.kde.org/p/mart/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@mart</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>what i would like to have logically splitted is the management of the floating property (and having positionItem()/releasePosition used around)</p>

<p>and on the other hand the signal handlers of minimumWidthChanged/widthChanged etc in Appletappearance, that's the biggest part</p></div>
</blockquote>

<p>Sorry, I don't understand what you would like me to do. Do you like me to split code to add additional commit where would be introduced managment of floating property? But why? it's useless without others bug 375307(fixes of work with layoutManager) fixes. And to split bug 375307 fixes from bug 375308(maximumSize handlers) is big work for me, because all my work centered over bug 375308. Other bugs were founded when debugging bug 375308 fixes. Because of that bug 375307 fixes is never were intended by me as standalone, they currently included only as additional bug fixing to bug 375308 fixes.</p>

<p>I can compress some code in handlers by adding function like</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">function setSizePropertyAndReposition(sizeProperty, newSizeProperty) {
    releasePosition();
    sizeProperty = newSizeProperty;
    positionItem();
    if (showAppletHandle && !handleMerged)
        appletHandle.positionHandle();
}</pre></div>

<p>Should I do it?</p>

<p>P.S. I understand that this code is not well. But in first place it couldn't be well because current state of plasmoid background code is in mess and it need to be rewritten. What I propose in this diff is temporary solution which fixes bugs until I or somebody else would rewrite plasmoid background to normal code.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D4204" rel="noreferrer">https://phabricator.kde.org/D4204</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>konstantinshtepa, Plasma<br /><strong>Cc: </strong>mart, davidedmundson, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>