<table><tr><td style="">broulik created this revision.<br />broulik added a reviewer: Plasma.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />broulik requested review of this revision.
</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/D26091">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>When a plasmoid collapses in Active state, the popup is hidden. In Passive state, however, one is returned to the hidden items list, as it cannot tell whether the plasmoid collapsed on its own or because the user clicked the same one in the hidden items list again.<br />
When we clear notifications, plasmoid state updates immediately as a result of model signals, and when we then try to collapse the plasmoid we're already hidden and just returned to the hidden items list.<br />
This patch addresses this by delaying the state change slightly. Futhermore, don't collapse the plasmoid when already hidden as that would just return the user to the hidden items list.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>I tried fixing that in System Tray but as I said there's no way for it to tell whether the plasmoid hid "on its own" or because the user clicked it again as the compact rep entirely governs this behavior when expanded. :/ It's ugly but I don't really see a better way.</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/D26091">https://phabricator.kde.org/D26091</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applets/notifications/package/contents/ui/FullRepresentation.qml<br />
applets/notifications/package/contents/ui/main.qml</div></div></div><br /><div><strong>To: </strong>broulik, Plasma<br /><strong>Cc: </strong>plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>