From a6cf71a1903557c7f610edfb2afc6228f7186977 Mon Sep 17 00:00:00 2001 From: purian23 Date: Sun, 1 Mar 2026 17:37:21 -0500 Subject: [PATCH] fix(notifications): Apply appIdSubs to iconFrImage fallback path - Consistent with the appIcon PR changes in #1880. --- .../Modules/Notifications/Center/HistoryNotificationCard.qml | 2 +- quickshell/Modules/Notifications/Center/NotificationCard.qml | 4 ++-- quickshell/Modules/Notifications/Popup/NotificationPopup.qml | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/quickshell/Modules/Notifications/Center/HistoryNotificationCard.qml b/quickshell/Modules/Notifications/Center/HistoryNotificationCard.qml index ef735e3d..95e74728 100644 --- a/quickshell/Modules/Notifications/Center/HistoryNotificationCard.qml +++ b/quickshell/Modules/Notifications/Center/HistoryNotificationCard.qml @@ -122,7 +122,7 @@ Rectangle { return ""; const appIcon = historyItem.appIcon; if (!appIcon) - return iconFromImage ? "image://icon/" + iconFromImage : ""; + return iconFromImage ? Paths.resolveIconUrl(iconFromImage) : ""; if (appIcon.startsWith("file://") || appIcon.startsWith("http://") || appIcon.startsWith("https://") || appIcon.includes("/")) return appIcon; if (appIcon.startsWith("material:") || appIcon.startsWith("svg:") || appIcon.startsWith("unicode:") || appIcon.startsWith("image:")) diff --git a/quickshell/Modules/Notifications/Center/NotificationCard.qml b/quickshell/Modules/Notifications/Center/NotificationCard.qml index 0fd23bf4..d7d53589 100644 --- a/quickshell/Modules/Notifications/Center/NotificationCard.qml +++ b/quickshell/Modules/Notifications/Center/NotificationCard.qml @@ -169,7 +169,7 @@ Rectangle { return ""; const appIcon = notificationGroup?.latestNotification?.appIcon; if (!appIcon) - return iconFromImage ? "image://icon/" + iconFromImage : ""; + return iconFromImage ? Paths.resolveIconUrl(iconFromImage) : ""; if (appIcon.startsWith("file://") || appIcon.startsWith("http://") || appIcon.startsWith("https://") || appIcon.includes("/")) return appIcon; if (appIcon.startsWith("material:") || appIcon.startsWith("svg:") || appIcon.startsWith("unicode:") || appIcon.startsWith("image:")) @@ -503,7 +503,7 @@ Rectangle { return ""; const appIcon = modelData?.appIcon; if (!appIcon) - return iconFromImage ? "image://icon/" + iconFromImage : ""; + return iconFromImage ? Paths.resolveIconUrl(iconFromImage) : ""; if (appIcon.startsWith("file://") || appIcon.startsWith("http://") || appIcon.startsWith("https://") || appIcon.includes("/")) return appIcon; if (appIcon.startsWith("material:") || appIcon.startsWith("svg:") || appIcon.startsWith("unicode:") || appIcon.startsWith("image:")) diff --git a/quickshell/Modules/Notifications/Popup/NotificationPopup.qml b/quickshell/Modules/Notifications/Popup/NotificationPopup.qml index b5b2d6ee..8c481ece 100644 --- a/quickshell/Modules/Notifications/Popup/NotificationPopup.qml +++ b/quickshell/Modules/Notifications/Popup/NotificationPopup.qml @@ -479,7 +479,7 @@ PanelWindow { return ""; const appIcon = notificationData.appIcon; if (!appIcon) - return iconFromImage ? "image://icon/" + iconFromImage : ""; + return iconFromImage ? Paths.resolveIconUrl(iconFromImage) : ""; if (appIcon.startsWith("file://") || appIcon.startsWith("http://") || appIcon.startsWith("https://") || appIcon.includes("/")) return appIcon; if (appIcon.startsWith("material:") || appIcon.startsWith("svg:") || appIcon.startsWith("unicode:") || appIcon.startsWith("image:"))