<table><tr><td style="">nicolasfella edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-b3pwfyxoq7nos42/">(Show Details)</a><br />nicolasfella edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-3f4ftbcaf2homqa/">(Show Details)</a><br />nicolasfella added reviewers: Frameworks, Plasma, broulik.
</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/D29021">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Alternative patch to D26605 and D26604<br />
<br />
KNotifications has proper support for all major platforms (freedesktop, windows, macOS, Android). The KPassivePopup fallback is only relevant for Linux when no FDO notification daemon is running. IMO the case of the user having no notification daemon at all is not worth supporting since the users intention is very clearly to not have notifications.<br />
<br />
Previously the KPassivePopup would sometimes appear when the server has crashed. Instead we now always make a call to the notification service, which then get's DBus-activated if it isn't running. In the Plasma case there is the plasma_waitforname helper that makes sure all notifications are captured until Plasma has restarted and forwards them to Plasma. This way no notifications are lost.<br />
<br />
This allows to drop a good amount of code. It reduces the dependency on Widgets so we can eventually get rid of it which is nice for Android. Furthermore the reduced complexity will make it easier to implement our plans for KF6</span></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Tested with new notification tester app.<br />
<br />
Killed Plasma, sent notification, restarted Plasma, notification showed up</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R289 KNotifications</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29021">https://phabricator.kde.org/D29021</a></div></div><br /><div><strong>To: </strong>nicolasfella, Frameworks, Plasma, broulik<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns<br /></div>