<table><tr><td style="">broulik created this revision.<br />broulik added a reviewer: Plasma.<br />broulik set the repository for this revision to rPLASMADESKTOP Plasma Desktop.<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/D1788" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>When there is only one virtual desktop, the pager becomes invisible. It makes no sense to update the models in this case.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Unfortunately root.visible evaluates true initially until I open and close the panelcontroller (the applet will be shown while configuring the panel) :( I have no clue why that happens. Other than that:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">No update when the pager is hidden</li>
<li class="remarkup-list-item">When opening panelcontroller immediately updates and stays live until the panelcontroller is closed again</li>
<li class="remarkup-list-item">Always updates when there's two or more desktops</li>
</ul>

<p>It's pretty terrible that the pager resets the models on every change, eg. even focus changes, causing all the window Rectangles to be destroyed and re-created, contributing to lags when opening popups. Until the thing is rewritten to use the new tasks model, this at least serves the default setup (where the pager will be added but just one virtual desktop).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPLASMADESKTOP Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D1788" rel="noreferrer">https://phabricator.kde.org/D1788</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applets/pager/package/contents/ui/main.qml<br />
applets/pager/plugin/pager.cpp<br />
applets/pager/plugin/pager.h</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, sebas<br /></div>