<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, hein, graesslin, VDG.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.<br />broulik 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/D9878" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Notifications do not show up on top of full screen windows to avoid them showing over e.g. a presentation.<br />
To keep you from missing anything that's been going on, keep the popup open until the full screen window goes away.<br />
There will be at most three popups shown at once, as has always been the case, and they would then cycle through.<br />
I chose to just keep the popups open rather than using the glowing notification icon as in the original patch for simplicity and to not introduce yet another "history" of notifications (persistent + the "history" + fullscreen missed ones).<br />
While at it simplify the "should notification close" handling by binding all conditions to a single binding</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>This basically resurrects <a href="https://git.reviewboard.kde.org/r/121010/" class="remarkup-link" target="_blank" rel="noreferrer">https://git.reviewboard.kde.org/r/121010/</a> but makes use of the new libtaskmanager that should also work on Wayland and reduces roundtrips as the task manager applet is present anyway and we're sharing the backend</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Verified that making a window full screen stops notification timer for all notifications</li>
<li class="remarkup-list-item">Verified that minimizing that window restarts the timer</li>
<li class="remarkup-list-item">Verified that switching virtual desktops restarts the timer if the window isnt on the new desktop</li>
<li class="remarkup-list-item">Verified that switching activities restarts the timer if the window isnt on the new activity, stopping an activity that causes a window to get moved to the current one isn't detected but that looks like a libTM bug</li>
<li class="remarkup-list-item">Verified that hovering a notification still stops that particular notification from disappearing</li>
<li class="remarkup-list-item">Verified that dragging a screenshot off a notification still stops that particular notification from disappearing</li>
<li class="remarkup-list-item">Didn't actually test multi screen but it's just using the same libtm filtering logic per screen there</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9878" rel="noreferrer">https://phabricator.kde.org/D9878</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applets/notifications/package/contents/ui/NotificationPopup.qml<br />
applets/notifications/package/contents/ui/Notifications.qml</div></div></div><br /><div><strong>To: </strong>broulik, Plasma, hein, graesslin, VDG<br /><strong>Cc: </strong>plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>