Skip to content

Commit

Permalink
Do not create dismiss links for notifications as these are not used a…
Browse files Browse the repository at this point in the history
…nyway

Signed-off-by: Claudio Cambra <[email protected]>
  • Loading branch information
claucambra committed Aug 4, 2023
1 parent 1293106 commit c975d40
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions src/gui/tray/notificationhandler.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -149,14 +149,6 @@ void ServerNotificationHandler::slotNotificationsReceived(const QJsonDocument &j
a._links.insert(al._primary? 0 : a._links.size(), al);
}

if (a._links.isEmpty()) {
ActivityLink dismissLink;
dismissLink._label = tr("Dismiss");
dismissLink._verb = "DELETE";
dismissLink._primary = false;
a._links.insert(0, dismissLink);
}

QUrl link(json.value("link").toString());
if (!link.isEmpty()) {
if (link.host().isEmpty()) {
Expand Down

1 comment on commit c975d40

@pancake-tongs
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why were notification dismissal buttons removed? This is a regression of issue 5585!
#5585

Please sign in to comment.