Skip to content

Commit

Permalink
Fix default mpris icon size (#370)
Browse files Browse the repository at this point in the history
* 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
  • Loading branch information
abmantis authored Mar 9, 2024
1 parent 54a4c5d commit ed1b721
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/controlCenter/widgets/mpris/mpris_player.vala
Original file line number Diff line number Diff line change
Expand Up @@ -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 () {
Expand Down Expand Up @@ -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);
}
}

Expand Down

0 comments on commit ed1b721

Please sign in to comment.