Bump version to 4.0.2

data/ui/library-panel.ui

@@ -221,10 +221,14 @@
       <property name="child">
         <object class="GtkBox">
           <property name="orientation">vertical</property>
+          <property name="vexpand">true</property>
+          <property name="hexpand">true</property>
           <child>
             <object class="GtkPicture">
               <property name="content-fit">contain</property>
               <property name="can-shrink">false</property>
+              <property name="vexpand">true</property>
+              <property name="hexpand">true</property>
               <binding name="tooltip-markup">
                 <lookup name="tooltip" type="GridItem">
                   <lookup name="item">GtkListItem</lookup>

meson.build

@@ -1,5 +1,5 @@
 project('mcg',
-    version: '4.0.1',
+    version: '4.0.2',
     meson_version: '>= 0.59.0',
     default_options: [
         'warning_level=2',

src/coverpanel.py

@@ -236,6 +236,7 @@ class CoverPanel(Gtk.Overlay):
         pixbuf = self._cover_pixbuf
         # Check pixelbuffer
         if pixbuf is None:
+            self.cover_default.set_pixel_size(min(size_width, size_height)/2)
             return
 
         # Skalierungswert für Breite und Höhe ermitteln
@@ -251,4 +252,5 @@ class CoverPanel(Gtk.Overlay):
             return
         self.cover_image.set_from_pixbuf(
             pixbuf.scale_simple(width, height, GdkPixbuf.InterpType.HYPER))
+        self.cover_image.set_pixel_size(min(width, height))
         self.cover_image.show()

src/librarypanel.py

@@ -293,15 +293,13 @@ class LibraryPanel(Adw.Bin):
 
     def set_albumart(self, album, data):
         if album in self._selected_albums:
+            self._standalone_pixbuf = None
             if data:
                 # Load image and draw it
                 try:
                     self._standalone_pixbuf = Utils.load_pixbuf(data)
                 except Exception:
                     self._logger.exception("Failed to set albumart")
-                    self._standalone_pixbuf = self._get_default_image()
-            else:
-                self._standalone_pixbuf = self._get_default_image()
             # Show image
             GObject.idle_add(self._show_image)
 
@@ -335,23 +333,24 @@ class LibraryPanel(Adw.Bin):
         self._grid_pixbufs.clear()
         for album_id in albums.keys():
             album = albums[album_id]
+            grid_item = GridItem(album)
+
             pixbuf = None
             try:
                 pixbuf = Utils.load_thumbnail(cache, self._client, album, size)
             except client.CommandException:
                 # Exception is handled by client
                 pass
-            except Exception as e:
+            except Exception:
-                self._logger.exception("Failed to load albumart", e)
+                self._logger.exception("Failed to load albumart")
             if pixbuf is None:
-                pixbuf = self._icon_theme.lookup_icon(
+                icon = self._get_default_icon(self._item_size, self._item_size)
-                    Utils.STOCK_ICON_DEFAULT, None, self._item_size,
+                grid_item.set_icon(icon)
-                    self._item_size, Gtk.TextDirection.LTR,
+            else:
-                    Gtk.IconLookupFlags.FORCE_SYMBOLIC)
-            if pixbuf is not None:
                 self._grid_pixbufs[album.get_id()] = pixbuf
-                GObject.idle_add(self._library_grid_model.append,
+                grid_item.set_cover(pixbuf)
-                                 GridItem(album, pixbuf))
+
+            GObject.idle_add(self._library_grid_model.append, grid_item)
 
             i += 1
             GObject.idle_add(self.progress_bar.set_fraction, i / n)
@@ -440,27 +439,26 @@ class LibraryPanel(Adw.Bin):
         pixbuf = self._standalone_pixbuf
         # Check pixelbuffer
         if pixbuf is None:
+            icon = self._get_default_icon(size_width, size_height)
+            self.standalone_image.set_from_paintable(icon)
+            self.standalone_image.set_pixel_size(min(size_width, size_height)/2)
             return
 
-        # Skalierungswert für Breite und Höhe ermitteln
+        (width, height) = Utils.calculate_size(pixbuf.get_width(),
-        ratio_w = float(size_width) / float(pixbuf.get_width())
+                                               pixbuf.get_height(), size_width,
-        ratio_h = float(size_height) / float(pixbuf.get_height())
+                                               size_height)
-        # Kleineren beider Skalierungswerte nehmen, nicht Hochskalieren
-        ratio = min(ratio_w, ratio_h)
-        ratio = min(ratio, 1)
-        # Neue Breite und Höhe berechnen
-        width = int(math.floor(pixbuf.get_width() * ratio))
-        height = int(math.floor(pixbuf.get_height() * ratio))
         if width <= 0 or height <= 0:
             return
         # Pixelpuffer auf Oberfläche zeichnen
         self.standalone_image.set_from_pixbuf(
             pixbuf.scale_simple(width, height, GdkPixbuf.InterpType.HYPER))
+        self.standalone_image.set_pixel_size(min(width, height))
         self.standalone_image.show()
 
-    def _get_default_image(self):
+    def _get_default_icon(self, width, height):
         return self._icon_theme.lookup_icon(Utils.STOCK_ICON_DEFAULT, None,
-                                            512, 512, Gtk.TextDirection.LTR,
+                                            width, height,
+                                            Gtk.TextDirection.LTR,
                                             Gtk.IconLookupFlags.FORCE_SYMBOLIC)
 
     def _get_selected_albums(self):

src/playlistpanel.py

@@ -171,15 +171,13 @@ class PlaylistPanel(Adw.Bin):
 
     def set_albumart(self, album, data):
         if album in self._selected_albums:
+            self._standalone_pixbuf = None
             if data:
                 # Load image and draw it
                 try:
                     self._standalone_pixbuf = Utils.load_pixbuf(data)
                 except Exception:
                     self._logger.exception("Failed to set albumart")
-                    self._cover_pixbuf = self._get_default_image()
-            else:
-                self._cover_pixbuf = self._get_default_image()
             # Show image
             GObject.idle_add(self._show_image)
 
@@ -197,6 +195,8 @@ class PlaylistPanel(Adw.Bin):
 
         cache = client.MCGCache(host, size)
         for album in playlist:
+            grid_item = GridItem(album)
+
             pixbuf = None
             # Load albumart thumbnail
             try:
@@ -207,12 +207,12 @@ class PlaylistPanel(Adw.Bin):
             except Exception:
                 self._logger.exception("Failed to load albumart")
             if pixbuf is None:
-                pixbuf = self._icon_theme.lookup_icon(
+                icon = self._get_default_icon(self._item_size, self._item_size)
-                    Utils.STOCK_ICON_DEFAULT, None, self._item_size,
+                grid_item.set_icon(icon)
-                    self._item_size, Gtk.TextDirection.LTR,
+            else:
-                    Gtk.IconLookupFlags.FORCE_SYMBOLIC)
+                grid_item.set_cover(pixbuf)
-            if pixbuf is not None:
+
-                self._playlist_grid_model.append(GridItem(album, pixbuf))
+            GObject.idle_add(self._playlist_grid_model.append, grid_item)
 
             if self._playlist_stop.is_set():
                 self._playlist_lock.release()
@@ -247,27 +247,26 @@ class PlaylistPanel(Adw.Bin):
         pixbuf = self._standalone_pixbuf
         # Check pixelbuffer
         if pixbuf is None:
+            icon = self._get_default_icon(size_width, size_height)
+            self.standalone_image.set_from_paintable(icon)
+            self.standalone_image.set_pixel_size(min(size_width, size_height)/2)
             return
 
-        # Skalierungswert für Breite und Höhe ermitteln
+        (width, height) = Utils.calculate_size(pixbuf.get_width(),
-        ratio_w = float(size_width) / float(pixbuf.get_width())
+                                               pixbuf.get_height(), size_width,
-        ratio_h = float(size_height) / float(pixbuf.get_height())
+                                               size_height)
-        # Kleineren beider Skalierungswerte nehmen, nicht Hochskalieren
-        ratio = min(ratio_w, ratio_h)
-        ratio = min(ratio, 1)
-        # Neue Breite und Höhe berechnen
-        width = int(math.floor(pixbuf.get_width() * ratio))
-        height = int(math.floor(pixbuf.get_height() * ratio))
         if width <= 0 or height <= 0:
             return
         # Pixelpuffer auf Oberfläche zeichnen
         self.standalone_image.set_from_pixbuf(
             pixbuf.scale_simple(width, height, GdkPixbuf.InterpType.HYPER))
+        self.standalone_image.set_pixel_size(min(width, height))
         self.standalone_image.show()
 
-    def _get_default_image(self):
+    def _get_default_icon(self, width, height):
         return self._icon_theme.lookup_icon(Utils.STOCK_ICON_DEFAULT, None,
-                                            512, 512, Gtk.TextDirection.LTR,
+                                            width, height,
+                                            Gtk.TextDirection.LTR,
                                             Gtk.IconLookupFlags.FORCE_SYMBOLIC)
 
     def _get_selected_albums(self):

src/utils.py

@@ -2,7 +2,9 @@
 
 import gi
 import hashlib
+import math
 import locale
+import logging
 import os
 
 gi.require_version('Gtk', '4.0')
@@ -35,9 +37,17 @@ class Utils:
             if albumart:
                 pixbuf = Utils.load_pixbuf(albumart)
             if pixbuf is not None:
-                pixbuf = pixbuf.scale_simple(size, size,
+                (width, height) = Utils.calculate_size(pixbuf.get_width(),
+                                                       pixbuf.get_height(),
+                                                       size, size)
+                pixbuf = pixbuf.scale_simple(width, height,
                                              GdkPixbuf.InterpType.HYPER)
-                pixbuf.savev(cache_url, 'jpeg', [], [])
+                try:
+                    pixbuf.savev(cache_url, 'jpeg', [], [])
+                except Exception as e:
+                    logger = logging.getLogger(__name__)
+                    logger.warning("Failed to save thumbnail for album\"%s\": "
+                        "%s", album.get_title(), e)
         return pixbuf
 
     @staticmethod
@@ -72,6 +82,19 @@ class Utils:
             m.update(value.encode('utf-8'))
         return m.hexdigest()
 
+    @staticmethod
+    def calculate_size(src_width, src_height, dest_width, dest_height):
+        ratio_w = float(dest_width) / float(src_width)
+        ratio_h = float(dest_height) / float(src_height)
+        ratio = min(min(ratio_w, ratio_h), 1)
+        if ratio == 1:
+            return (src_width, src_height)
+
+        width = int(math.floor(src_width * ratio))
+        height = int(math.floor(src_height * ratio))
+
+        return (width, height)
+
 
 class SortOrder:
     ARTIST = 0
@@ -86,11 +109,9 @@ class GridItem(GObject.GObject):
     tooltip = GObject.Property(type=str, default=None)
     cover = GObject.Property(type=Gdk.Paintable, default=None)
 
-    def __init__(self, album, cover):
+    def __init__(self, album):
         super().__init__()
         self._album = album
-        if cover:
-            self.cover = Gdk.Texture.new_for_pixbuf(cover)
         self.tooltip = GObject.markup_escape_text("\n".join([
             album.get_title(), ', '.join(album.get_dates()),
             Utils.create_artists_label(album),
@@ -103,6 +124,9 @@ class GridItem(GObject.GObject):
     def set_cover(self, cover):
         self.cover = Gdk.Texture.new_for_pixbuf(cover)
 
+    def set_icon(self, icon):
+        self.cover = icon
+
 
 class SearchFilter(Gtk.Filter):