<table><tr><td style="">mart 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/D11410">View Revision</a></tr></table><br /><div><div><p>it's moving on the right track.. i would like the checkbox to appear like the one in the panel spacer, in the context menu could be tricky as would need to inject into the applet's contextualactons, so let's forget about it for now.</p>

<p>one thing that could be done is to put that button in the applet handle that appears on hover when the panel controller is open, with the same icon of the add panel spacer in a checkable toolbutton (just icon, tooltip on hover) the panel spacer should use the same config ui.</p>

<p>i would like to expose this ui only for applets that want to expand, and not for those that are already compact (i don't think this option would make any sense in kicoff for instance)</p>

<p>now, on the configuration side.. it's interesting the approach you took with a list of ids in the panel config, i tought more on something directly down in libplsma's applet, but this stays more confined to the panel, so i like it.</p></div></div><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-56582">View Inline</a><span style="color: #4b4d51; font-weight: bold;">main.xml:13</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #00702a"><entry</span> <span style="color: #354bb3">name=</span><span style="color: #766510">"fixedSizeOverride"</span> <span style="color: #354bb3">type=</span><span style="color: #766510">"IntList"</span><span style="color: #00702a">></span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">      <span style="color: #00702a"><label></span>list of items for which we set fillWidth = false<span style="color: #00702a"></label></span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #00702a"></entry></span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">list of applet ids that should not have an expanding size policy to save space</p></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, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>