FdoNotificationDaemon: Don't destroy all non-resident on app activation

Don't close all non-resident notifications if one notification of an app
is activated with the default action. This behavior isn't part of the
fdo notification specs and IMO it may confuse app developers and user
about why notifications are removed.

Additionally, with grouping by app, add in a future commit it's much
easier to dismiss all notifications of an app.
This commit is contained in:
Julian Sparber 2024-05-17 11:40:49 +02:00
parent e889ee1983
commit 088c2035c7
2 changed files with 0 additions and 8 deletions

View file

@ -597,13 +597,6 @@ export const Source = GObject.registerClass({
// To be overridden by subclasses
open() {
}
destroyNonResidentNotifications() {
for (let i = this.notifications.length - 1; i >= 0; i--) {
if (!this.notifications[i].resident)
this.notifications[i].destroy();
}
}
});
SignalTracker.registerDestroyableType(Source);

View file

@ -368,7 +368,6 @@ class FdoNotificationDaemonSource extends MessageTray.Source {
open() {
this.openApp();
this.destroyNonResidentNotifications();
}
openApp() {