From 8fed0b83d085d49661185b50f98f3e8f02ee681d Mon Sep 17 00:00:00 2001
From: Julian Sparber <julian@sparber.net>
Date: Tue, 19 Dec 2023 16:12:59 +0100
Subject: [PATCH] notification: Show symbolic icons in a circle and smaller

Huge symbolic icons look bad therefore we special case them, this
obviously can only work for icons that use the system icons.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/3066>
---
 data/theme/gnome-shell-sass/widgets/_message-list.scss | 8 ++++++++
 js/ui/calendar.js                                      | 9 +++++----
 2 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/data/theme/gnome-shell-sass/widgets/_message-list.scss b/data/theme/gnome-shell-sass/widgets/_message-list.scss
index 10fa88258..30fe4ad00 100644
--- a/data/theme/gnome-shell-sass/widgets/_message-list.scss
+++ b/data/theme/gnome-shell-sass/widgets/_message-list.scss
@@ -137,6 +137,14 @@
       > StIcon {
         icon-size: $base_icon_size * 3; // 48px
         -st-icon-style: symbolic;
+
+        .message-themed-icon {
+          border-radius: $forced_circular_radius; // is circular
+          background-color: transparentize($fg_color, 0.8);
+          icon-size: $base_icon_size;
+          min-width: $base_icon_size * 3;
+          min-height: $base_icon_size * 3;
+        }
       }
     }
 
diff --git a/js/ui/calendar.js b/js/ui/calendar.js
index 455cfa184..c43290ac7 100644
--- a/js/ui/calendar.js
+++ b/js/ui/calendar.js
@@ -790,10 +790,11 @@ class NotificationMessage extends MessageList.Message {
     }
 
     _getIcon() {
-        if (this.notification.gicon) {
-            return new St.Icon({
-                gicon: this.notification.gicon,
-            });
+        const {gicon} = this.notification;
+        if (gicon) {
+            const styleClass =
+                gicon instanceof Gio.ThemedIcon ? 'message-themed-icon' : '';
+            return new St.Icon({gicon, styleClass});
         } else {
             return null;
         }