Commit 08b027f1 authored by Kai Uwe Broulik's avatar Kai Uwe Broulik 🍇

[Notifications] Group applications with same display name but no desktop entry

The specification says appName should be the display name of the application.
An application may not send a desktop name at which point we have no other way of identifying it
than the user-visible name it provides us.

Differential Revision: https://phabricator.kde.org/D21460
parent 143969f2
......@@ -44,7 +44,7 @@ bool NotificationGroupingProxyModel::appsMatch(const QModelIndex &a, const QMode
const QString bDesktopEntry = b.data(Notifications::DesktopEntryRole).toString();
return !aName.isEmpty() && aName == bName
&& !aDesktopEntry.isEmpty() && aDesktopEntry == bDesktopEntry;
&& aDesktopEntry == bDesktopEntry;
}
bool NotificationGroupingProxyModel::isGroup(int row) const
......
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