<table><tr><td style="">ngraham created this revision.<br />ngraham added reviewers: apol, Discover Software Store.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />ngraham 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/D22429">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Discover's "Updates are available" notification currently suffers from some problems:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">With the default timeout, it disappears quickly and is likely to be missed</li>
<li class="remarkup-list-item">After timing out, it appears in thie history, where is is both redundant (because the Update Notifier system tray icon has become visible) and also useless (because it can't be interacted with due to <a href="https://bugs.kde.org/show_bug.cgi?id=407361" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=407361</a> and <a href="https://bugs.kde.org/show_bug.cgi?id=407667" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=407667</a>)</li>
<li class="remarkup-list-item">Once updates are performed, the notification in the history sticks around uselessly because notifications can't be revoked once they're in the history (See <a href="https://bugs.kde.org/show_bug.cgi?id=409323#c3" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=409323#c3</a>)</li>
</ol>

<p>This patch fixes those issues by making the update persistent but low priority.</p>

<p>This means it must be noticed by the user and interacted with in some way before it<br />
disappears from the screen--either by dismissing it or running the updates. But once<br />
dismissed, it's gone forever, and doesn't clutter up your history.</p>

<p>BUG: 409757<br />
BUG: 409331<br />
FIXED-IN: 5.17.0</p></div></div><br /><div><strong>TEST PLAN</strong><div><ol class="remarkup-list">
<li class="remarkup-list-item">Have updates available</li>
<li class="remarkup-list-item">Deploy patch</li>
<li class="remarkup-list-item">Restart plasmashell</li>
<li class="remarkup-list-item">See that the notification is now persistent</li>
<li class="remarkup-list-item">Dismiss notification</li>
<li class="remarkup-list-item">See that it no longer clogs up the history</li>
<li class="remarkup-list-item">Restart plasmashell again</li>
<li class="remarkup-list-item">Click on the notification's body or "Update" button</li>
<li class="remarkup-list-item">See that Discover opens and the notification disappears</li>
</ol></div></div><br /><div><strong>REPOSITORY</strong><div><div>R134 Discover Software Store</div></div></div><br /><div><strong>BRANCH</strong><div><div>polish-notification-behavior (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22429">https://phabricator.kde.org/D22429</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>libdiscover/resources/discoverabstractnotifier.notifyrc<br />
notifier/DiscoverNotifier.cpp<br />
notifier/DiscoverNotifier.h</div></div></div><br /><div><strong>To: </strong>ngraham, apol, Discover Software Store<br /><strong>Cc: </strong>plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>