<table><tr><td style="">Pitel updated this revision to Diff 29940.<br />Pitel 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><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/D11410#229277" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D11410#229277</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;">@mart</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><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></div>
</blockquote>

<p>I'm not sure that is good idea: There are already some items injected in context menus (e.g. Unlock widgets action) and looking at my panel some applets' context menus are not injected. I believe it is because they have multiple menus bound to specific parts of itself and only applet's global menu is (and can) be injected. Most notable examples are systray except the expansion arrow and task icon manager except empty space (if it has set <tt style="background: #ebebeb; font-size: 13px;">fixedWidth</tt> I was unable to find a place in it which would show me context menu containing Unlock widget action).</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><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></blockquote>

<p>The combox is currently in the popup showing on hover when panel controller is open. I can make it replace spacer's specific option. I can just remove spacer's option from its context menu and add some temporary code to convert spacer's option into the new one. (And a few version later we can drop this code.) Is it what you meant or did I misunderstand you?</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><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></blockquote>

<p>Done.</p>

<p>Changes:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Fix config item label</li>
<li class="remarkup-list-item">Show checkbox only for expanding applets</li>
<li class="remarkup-list-item">A bit polish checkbox placement</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D11410?vs=29745&id=29940">https://phabricator.kde.org/D11410?vs=29745&id=29940</a></div></div><br /><div><strong>BRANCH</strong><div><div>fixedSizeOverride</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>AFFECTED FILES</strong><div><div>containments/panel/contents/config/main.xml<br />
containments/panel/contents/ui/ConfigOverlay.qml<br />
containments/panel/contents/ui/main.qml</div></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>