Skip to content

Commit

Permalink
[transfer-engine] Update notification icon configuration. JB#50444
Browse files Browse the repository at this point in the history
Also use same icon for all notifications.
  • Loading branch information
Bea Lam committed Jul 20, 2020
1 parent 96fff9d commit 72b5bc3
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions src/transferengine.cpp
Expand Up @@ -335,8 +335,7 @@ void TransferEnginePrivate::sendNotification(TransferEngineData::TransferType ty
QString previewSummary;
bool useProgress = false;
Notification::Urgency urgency = Notification::Normal;
QString appIcon = QStringLiteral("icon-lock-information");
QString icon = QStringLiteral("icon-lock-transfer");
QString appIcon = 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 @@ -383,9 +382,7 @@ void TransferEnginePrivate::sendNotification(TransferEngineData::TransferType ty

} else if (status == TransferEngineData::TransferInterrupted) {
urgency = Notification::Critical;
appIcon = QStringLiteral("icon-lock-information");
category = TRANSFER_ERROR_EVENT_CATEGORY;
icon.clear();

switch (type) {
case TransferEngineData::Upload:
Expand Down Expand Up @@ -468,8 +465,8 @@ void TransferEnginePrivate::sendNotification(TransferEngineData::TransferType ty
notification.setPreviewBody(previewBody);
notification.setUrgency(urgency);

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

if (useProgress) {
Expand Down

0 comments on commit 72b5bc3

Please sign in to comment.