<table><tr><td style="">broulik updated this revision to Diff 71877.<br />broulik retitled this revision from "[Notifications] Add hint for showing a notification even in do not disturb mode" to "[Notifications] Add hint for a notification being response to explicit user interaction".<br />broulik edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-wrvai4uzksxyrhf/">(Show Details)</a><br />broulik edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-77rwl7l4vlbg5pe/">(Show Details)</a><br />broulik added a comment.
</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/D25935">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">Make more generic and implicit</li>
</ul></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>It only overrules filtering based on urgency, disabling notifications for that application altogether will still hide them.<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
Such a notification also won't increase the unread counter as it can be assumed when they do something, e.g. click on "Pair", they will notice that the pairing progress dialog went away and a confirmation notification is shown.</span></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;">Enabled dnd mode, `notify-send foo bar -h string:x-kde-<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">show-when-inhibited</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">user-action</span>:true` and it showed up regardless<br />
</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D25935?vs=71398&id=71877">https://phabricator.kde.org/D25935?vs=71398&id=71877</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25935">https://phabricator.kde.org/D25935</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>libnotificationmanager/notification.cpp<br />
libnotificationmanager/notification.h<br />
libnotificationmanager/notification_p.h<br />
libnotificationmanager/notificationfilterproxymodel.cpp<br />
libnotificationmanager/notifications.h<br />
libnotificationmanager/notificationsmodel.cpp</div></div></div><br /><div><strong>To: </strong>broulik, Plasma<br /><strong>Cc: </strong>ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>