<table><tr><td style="">mart accepted this revision.<br />mart added a reviewer: mart.<br />mart added a comment.<br />This revision is now accepted and ready to land.
</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/D4142" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D4142#77864" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D4142#77864</a>, <a href="https://phabricator.kde.org/p/albertvaka/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@albertvaka</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>No, there is no change in our UI at all. In Plasma and KDE apps we can decide to never use this feature if we think that the UX is better that way, but IMO this is a generic (not Plasma-specific) framework and it should provide access to the functionality anyway. This is a really common use case for notifications on most</p></div>
</blockquote>

<p>for the framework part, as i said, the code is OK, so go for it..<br />
but, to me it really strongly begs the question whether we in the end want this exposed in the UI or not.<br />
having the functionality entering the framework is a good occasion to think about it and take a decision.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R289 KNotifications</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D4142" rel="noreferrer">https://phabricator.kde.org/D4142</a></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>albertvaka, Frameworks, VDG, mart<br /><strong>Cc: </strong>colomar, broulik, mart, Frameworks<br /></div>