<table><tr><td style="">nicolasfella created this revision.<br />nicolasfella added reviewers: Frameworks, broulik.<br />Herald added a project: Frameworks.<br />Herald added a subscriber: kde-frameworks-devel.<br />nicolasfella 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/D26605">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>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. <a href="https://phabricator.kde.org/D26604" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D26604</a> addresses the corner case of a crashing notification daemon. IMO the case of the user having no notification daemon at all is not worth supporting since the users intention is verly clearly to not have notifications.</p>

<p>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</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Still get regular notifications</p>

<p>Depends on <a href="https://phabricator.kde.org/D26604" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D26604</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R289 KNotifications</div></div></div><br /><div><strong>BRANCH</strong><div><div>nofallback</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26605">https://phabricator.kde.org/D26605</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/notifybypopup.cpp<br />
src/notifybypopup.h</div></div></div><br /><div><strong>To: </strong>nicolasfella, Frameworks, broulik<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>