Fix default mpris icon size (#370)

* Fix default mpris icon size The widget was was using the .ui size instead of the user config size in the following situations: - there was no media player app - the media player app icon was used - the audio-x-generic-symbolic icon was used This fix sets the default size for the album art icon according to the user's config. set_from_gicon and set_from_icon_name receive a Gtk.IconSize enum value, not an int, so we need to call set_pixel_size to set the proper size. * Use INVALID for IconSize * Fix lint

Fix default mpris icon size (#370)
ed1b721a · Abílio Costa · GitHub · 54a4c5d0 · ed1b721a
Unverified Commit ed1b721a authored Mar 09, 2024 by Abílio Costa Committed by GitHub Mar 09, 2024
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

mpris_player.vala src/controlCenter/widgets/mpris/mpris_player.vala +3 -2

No files found.
--- a/src/controlCenter/widgets/mpris/mpris_player.vala
+++ b/src/controlCenter/widgets/mpris/mpris_player.vala
@@ -96,6 +96,7 @@ namespace SwayNotificationCenter.Widgets.Mpris {
                    update_buttons (source.media_player.metadata);
                });
            });
+            album_art.set_pixel_size (mpris_config.image_size);
        }

        public void before_destroy () {
@@ -380,11 +381,11 @@ namespace SwayNotificationCenter.Widgets.Mpris {
                icon = desktop_entry.get_icon ();
            }
            if (icon != null) {
-                album_art.set_from_gicon (icon, mpris_config.image_size);
+                album_art.set_from_gicon (icon, Gtk.IconSize.INVALID);
            } else {
                // Default icon
                album_art.set_from_icon_name ("audio-x-generic-symbolic",
-                                              mpris_config.image_size);
+                                              Gtk.IconSize.INVALID);
            }
        }