From 26a53e75b4f18d50fdd44e58700ec926bc6095d3 Mon Sep 17 00:00:00 2001 From: Peter Fern Date: Thu, 11 Apr 2024 23:06:14 +1000 Subject: [PATCH] fix(notifications): We advertise notification markup support, enable it I previously had this disabled to workaround Thunderbird in particular, but many other applications correctly support markup, so we should either disable advertising it, or correctly enable it. Opted for the latter. --- cmd/hyprpanel-client/notification_item.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/cmd/hyprpanel-client/notification_item.go b/cmd/hyprpanel-client/notification_item.go index f63356a..2e5420f 100644 --- a/cmd/hyprpanel-client/notification_item.go +++ b/cmd/hyprpanel-client/notification_item.go @@ -149,9 +149,9 @@ func (i *notificationItem) build(container *gtk.Box) error { body := gtk.NewLabel(``) i.AddRef(body.Unref) body.SetSelectable(true) - // I'd love to be able to use SetMarkup here, but Thunderbird for example does not - // entity encode <> which results in empty notifications. - body.SetText(i.data.Body) + // Enable markup output, but beware that Thunderbird does not correclty encode their messages: + // https://bugzilla.mozilla.org/show_bug.cgi?id=1432209 + body.SetMarkup(i.data.Body) body.SetVexpand(true) body.SetWrap(true) body.SetWrapMode(pango.WrapWordCharValue)