<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, sitter.<br />broulik set the repository for this revision to rPLASMAWORKSPACE 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/D2044" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This ensures the logout dialog is kept in memory and continues to work even after a Qt or other library update.</p>

<p>BUG: 364897</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Mostly works. I logged in and then screwed up my logout dialog QML file and still got a shutdown dialog.</p>

<p>We need to adjust the logout dialog QML though so it stops the timer when not visible (my root.visible approach didn't work) and resets the timeout. Also it doesn't seem to properly update the selected shutdown type, ie. if I select Logout and then select Shutdown it doesn't notice. Ideally, we'll just do that when we're re-designing the logout dialog anyway.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPLASMAWORKSPACE Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2044" rel="noreferrer">https://phabricator.kde.org/D2044</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>ksmserver/shutdown.cpp<br />
ksmserver/shutdowndlg.cpp<br />
ksmserver/shutdowndlg.h<br />
ksmserver/startup.cpp<br />
ksmserver/tests/test.cpp<br />
lookandfeel/contents/components/LogoutScreen.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, sitter<br /><strong>Cc: </strong>plasma-devel, jensreuterberg, sebas<br /></div>