Skip to content

Commit

Permalink
Fix notification icon not getting set
Browse files Browse the repository at this point in the history
Almost got there by earlier progress changes.
  • Loading branch information
pvuorela committed Mar 27, 2018
1 parent ad786e2 commit e1efa03
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/transferengine.cpp
Expand Up @@ -335,7 +335,7 @@ void TransferEnginePrivate::sendNotification(TransferEngineData::TransferType ty
bool useProgress = false;
Notification::Urgency urgency = Notification::Normal;
QString appIcon = QStringLiteral("icon-lock-information");
QString icon = QStringLiteral("x-nemo-icon=icon-lock-transfer");
QString icon = QStringLiteral("icon-lock-transfer");

// TODO: explicit grouping of transfer notifications is now removed, as grouping
// will now be performed by lipstick. We may need to reinstate group summary
Expand Down Expand Up @@ -467,6 +467,10 @@ void TransferEnginePrivate::sendNotification(TransferEngineData::TransferType ty
notification.setPreviewBody(previewBody);
notification.setUrgency(urgency);

if (!icon.isEmpty()) {
notification.setIcon(icon);
}

if (useProgress) {
notification.setHintValue(TRANSFER_PROGRESS_HINT, static_cast<double>(progress));
}
Expand Down

0 comments on commit e1efa03

Please sign in to comment.