<table><tr><td style="">Pitel added inline comments.
</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/D11410">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D11410#inline-58865">View Inline</a><span style="color: #4b4d51; font-weight: bold;">mart</span> wrote in <span style="color: #4b4d51; font-weight: bold;">ConfigOverlay.qml:385</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">this will probably need as well<br />
currentApplet.applet.pluginName == "org.kde.plasma.panelspacer"</p>

<p style="padding: 0; margin: 8px;">or it gets lost when the spacer is configured as not expanding and i guess the spacer is special and we want this option to be always there<br />
(step 2 would be removing the option in the spacer and make it use this system, but it's for a next commit)</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">It does not, it is already in definition of <tt style="background: #ebebeb; font-size: 13px;">expandingApplet</tt> (line 254 of <tt style="background: #ebebeb; font-size: 13px;">main.qml</tt>).</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D11410#inline-58866">View Inline</a><span style="color: #4b4d51; font-weight: bold;">mart</span> wrote in <span style="color: #4b4d51; font-weight: bold;">ConfigOverlay.qml:402</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">is it necessary to use a temp copy?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">My reasons for temp copy were:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item"><tt style="background: #ebebeb; font-size: 13px;">push</tt> method has no effect on <tt style="background: #ebebeb; font-size: 13px;">plasmoid.configuration.fixedSizeOverride</tt> (I assume this has something to do with the whole configuration magic behind but it might be worth further investigating.)</li>
<li class="remarkup-list-item">QML does not notify the change of property if the property is an array and you change only its elements (because the property still contains the same object) so we need to trigger this signal somehow.</li>
</ul></div></div></div></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/D11410">https://phabricator.kde.org/D11410</a></div></div><br /><div><strong>To: </strong>Pitel, Plasma<br /><strong>Cc: </strong>mart, hein, plasma-devel, ragreen, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>