From 49f884f1d23f4b9d8b93f1f69478ab09649b0af1 Mon Sep 17 00:00:00 2001 From: "Danilo M." Date: Tue, 15 Sep 2026 18:34:28 +0200 Subject: fix(notify): treat icons up to 256px as an app icon, not a preview A 256px themed icon or logo was still drawn as a large balloon preview. The bound is now 256 on the long side, which covers kitty's own PNG and typical application logos; a screenshot stays well above it. --- internal/notify/classify.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'internal/notify/classify.go') diff --git a/internal/notify/classify.go b/internal/notify/classify.go index c7e0dc8..d00a7fa 100644 --- a/internal/notify/classify.go +++ b/internal/notify/classify.go @@ -21,10 +21,10 @@ import ( "strings" ) -// IconMaxPixels is the long-side bound below which a content image is treated -// as an icon rather than a preview. A screenshot is far larger; a themed icon -// or an application logo is not. -const IconMaxPixels = 128 +// IconMaxPixels is the long-side bound at or below which a content image is +// treated as an icon rather than a preview. A themed icon or an application +// logo is 256 or less; a screenshot is far larger. +const IconMaxPixels = 256 // IsIconImage reports whether a content image should fill the app-icon slot // instead of the large preview. Clients disagree about where the icon goes: -- cgit v1.2.3