From 986b7741428e1f00a5b0688e254665eab90c025a Mon Sep 17 00:00:00 2001 From: Boyuan Yang <073plan@gmail.com> Date: Mon, 26 Mar 2018 19:17:24 +0800 Subject: [PATCH] systemnotification: Fix wrong icon name (#174) Current implementation of SystemNotification::sendMessage() is wrongly using "flameshot.png" as icon name. We should strip the suffix and directly use "flameshot" to comply with Desktop Notification Specification. This commit does so and indeed makes notifications show flameshot icon on GNOME Shell again. See also https://developer.gnome.org/notification-spec/ . --- src/utils/systemnotification.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/utils/systemnotification.cpp b/src/utils/systemnotification.cpp index dcccbffb..ed4a2738 100644 --- a/src/utils/systemnotification.cpp +++ b/src/utils/systemnotification.cpp @@ -37,7 +37,7 @@ void SystemNotification::sendMessage( QList args; args << (qAppName()) //appname << static_cast(0) //id - << "flameshot.png" //icon + << "flameshot" //icon << title //summary << text //body << QStringList() //actions