<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, afiestas.<br />broulik set the repository for this revision to rPOWERDEVIL Powerdevil.<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/D2325" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>A button press (technically the "lid closed" event is also a button press) should always be some consicious action triggered by the user. By emitting this signal unconditionally on profile load (ie. whenever changing an activity or (un)plugging AC) the session might always lock or suspend when this happens.</p>

<p>If and only if a profile would really like to take some action or set some state depending on the lid state on profile load - which none do, the handle button press action should only react on explicit button presses anyway - it can still ask the backend for isLidClosed in its onProfileLoad method.</p>

<p>BUG: 366125<br />
FIXED-IN: 5.7.4</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Closed the lid, still suspends. I can't find any evidence about the reason this was put in there in the first place.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPOWERDEVIL Powerdevil</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2325" rel="noreferrer">https://phabricator.kde.org/D2325</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>daemon/powerdevilcore.cpp</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, afiestas<br /><strong>Cc: </strong>plasma-devel, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>