<table><tr><td style="">broulik created this revision.<br />broulik added a reviewer: Plasma.<br />broulik set the repository for this revision to R120 Plasma Workspace.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.
</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/D4488" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Instead of calling it in response to any of the properties it depends on.<br />
This also saves some calls to this function on startup (80 instead of 90 for me with 14 tray icons) and keeps plasmoids from shifting around just because their status changed from e.g. Active to AcceptingInput (like touchpad when it asks for whether you really want to disable touchpad with no mouse attached) although the item's effective visibility didn't actually change.</p>

<p>BUG: 375112<br />
CCBUG: 365630<br />
FIXED-IN: 5.10.0</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Disabling categories still completely hides items</li>
<li class="remarkup-list-item">Active/passive change still moves items from compact rep to popup and vice-versa</li>
<li class="remarkup-list-item">when touchpad applet asks for confirmation it no longer shifts around (used to move to the left-most (after notifications) position)</li>
</ul>

<p>(I noticed that we don't actually support the <tt style="background: #ebebeb; font-size: 13px;">HiddenStatus</tt> for plasmoids anymore which was originally introduced for system tray for applets to completely hide themselves)</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/D4488" rel="noreferrer">https://phabricator.kde.org/D4488</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applets/systemtray/package/contents/ui/items/AbstractItem.qml<br />
applets/systemtray/package/contents/ui/main.qml</div></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>broulik, Plasma<br /><strong>Cc: </strong>plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>