Commit d9db05dd authored by Nate Graham's avatar Nate Graham 🔩

Revert change to make the notification persistent

Summary:
Nobody liked it and it was annoying. From a conceptual standpoint, it was redundant
anyway because the SNI in the system tray is already persistent when there's an update
available.

Targeting 5.17.1 for this.

Test Plan: Restart plasmashell, see that the "updates available" notification disappears after a few seconds

Reviewers: apol, broulik, #discover_software_store

Reviewed By: apol, #discover_software_store

Subscribers: ognarb, nicolasfella, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D24766
parent a3f71afe
......@@ -81,7 +81,7 @@ void DiscoverNotifier::showUpdatesNotification()
return;
}
m_updatesAvailableNotification = KNotification::event(QStringLiteral("Update"), message(), {}, iconName(), nullptr, KNotification::Persistent, QStringLiteral("discoverabstractnotifier"));
m_updatesAvailableNotification = KNotification::event(QStringLiteral("Update"), message(), {}, iconName(), nullptr, KNotification::CloseOnTimeout, QStringLiteral("discoverabstractnotifier"));
const QString name = i18n("Update");
m_updatesAvailableNotification->setDefaultAction(name);
m_updatesAvailableNotification->setActions({name});
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment