diff options
| author | Danilo M. <danix@danix.xyz> | 2026-09-15 18:34:28 +0200 |
|---|---|---|
| committer | Danilo M. <danix@danix.xyz> | 2026-09-15 18:34:28 +0200 |
| commit | 49f884f1d23f4b9d8b93f1f69478ab09649b0af1 (patch) | |
| tree | 84a146c7d139488b9952736910d4bfc9ed3e61fd /internal/notify/classify.go | |
| parent | 8087086bc0a996e7dc020ba8bcf1332dd843f034 (diff) | |
| download | notifyd-49f884f1d23f4b9d8b93f1f69478ab09649b0af1.tar.gz notifyd-49f884f1d23f4b9d8b93f1f69478ab09649b0af1.zip | |
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.
Diffstat (limited to 'internal/notify/classify.go')
| -rw-r--r-- | internal/notify/classify.go | 8 |
1 files changed, 4 insertions, 4 deletions
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: |