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/ .
This commit is contained in:
Boyuan Yang
2018-03-26 19:17:24 +08:00
committed by Dharkael
parent 38d08504eb
commit 986b774142

View File

@@ -37,7 +37,7 @@ void SystemNotification::sendMessage(
QList<QVariant> args;
args << (qAppName()) //appname
<< static_cast<unsigned int>(0) //id
<< "flameshot.png" //icon
<< "flameshot" //icon
<< title //summary
<< text //body
<< QStringList() //actions