From 9a22de4954a55252c267b74de82544bd69e5a241 Mon Sep 17 00:00:00 2001
From: 2 * r + 2 * t <61896496+soramanew@users.noreply.github.com>
Date: Mon, 13 Jan 2025 14:04:19 +1100
Subject: notifpopups: fix app icon

For some reason firefox gives its desktop entry with as Firefox while the icon is lowercase, so use subs
Also use icon lookups to check so no broken icons
---
 modules/notifpopups.tsx | 20 +++++++++++++++-----
 1 file changed, 15 insertions(+), 5 deletions(-)

(limited to 'modules/notifpopups.tsx')

diff --git a/modules/notifpopups.tsx b/modules/notifpopups.tsx
index f83d9de..8b95f98 100644
--- a/modules/notifpopups.tsx
+++ b/modules/notifpopups.tsx
@@ -1,6 +1,7 @@
 import { GLib, register, timeout } from "astal";
 import { Astal, Gtk, Widget } from "astal/gtk3";
 import AstalNotifd from "gi://AstalNotifd";
+import { desktopEntrySubs } from "../utils/icons";
 
 const urgencyToString = (urgency: AstalNotifd.Urgency) => {
     switch (urgency) {
@@ -23,7 +24,18 @@ const getTime = (time: number) => {
     return messageTime.format("%d/%m");
 };
 
-const Icon = ({ icon }: { icon: string }) => {
+const AppIcon = ({ appIcon, desktopEntry }: { appIcon: string; desktopEntry: string }) => {
+    // Try app icon
+    let icon = Astal.Icon.lookup_icon(appIcon) && appIcon;
+    // Try desktop entry
+    if (!icon) {
+        if (desktopEntrySubs.hasOwnProperty(desktopEntry)) icon = desktopEntrySubs[desktopEntry];
+        else if (Astal.Icon.lookup_icon(desktopEntry)) icon = desktopEntry;
+    }
+    return icon ? <icon className="app-icon" icon={icon} /> : null;
+};
+
+const Image = ({ icon }: { icon: string }) => {
     if (GLib.file_test(icon, GLib.FileTest.EXISTS))
         return (
             <box
@@ -51,9 +63,7 @@ class NotifPopup extends Widget.Box {
                 <box className="wrapper">
                     <box vertical className={`popup ${urgencyToString(notification.urgency)}`}>
                         <box className="header">
-                            {(notification.appIcon || notification.desktopEntry) && (
-                                <icon className="app-icon" icon={notification.appIcon || notification.desktopEntry} />
-                            )}
+                            <AppIcon appIcon={notification.appIcon} desktopEntry={notification.appName} />
                             <label className="app-name" label={notification.appName ?? "Unknown"} />
                             <box hexpand />
                             <label
@@ -66,7 +76,7 @@ class NotifPopup extends Widget.Box {
                         </box>
                         <box hexpand className="separator" />
                         <box className="content">
-                            {notification.image && <Icon icon={notification.image} />}
+                            {notification.image && <Image icon={notification.image} />}
                             <box vertical>
                                 <label className="summary" xalign={0} label={notification.summary} truncate />
                                 <label className="body" xalign={0} label={notification.body} wrap useMarkup />
-- 
cgit v1.2.3-freya

