Merge pull request #11 from quexten/feature/notification-icon-optional

rymdport · Apr 30, 2024 · a9536c1 · a9536c1
2 parents 868abfe + a21edbe
commit a9536c1
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/notification/notification.go b/notification/notification.go
@@ -44,7 +44,11 @@ func Add(id uint, content Content) error {
 	data := map[string]dbus.Variant{
 		"title": convert.FromString(content.Title),
 		"body":  convert.FromString(content.Body),
-		"icon":  convert.FromString(content.Icon),
+	}
+
+	// Only add the icon field when it is set
+	if content.Icon != "" {
+		data["icon"] = convert.FromString(content.Icon)
 	}
 
 	// Only add the priority field when it is set.