fix: refactor mediaBrowserItemsPerRow, try 2

punxaphil · Sep 18, 2024 · 03d4e19 · 03d4e19
1 parent 54d525d
commit 03d4e19
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/card.ts b/src/card.ts
@@ -213,7 +213,8 @@ export class Card extends LitElement {
     } else {
       this.section = PLAYER;
     }
-    newConfig.mediaBrowserItemsPerRow = newConfig.mediaBrowserItemsPerRow || 4;
+    const itemsPerRow = parseInt(newConfig.mediaBrowserItemsPerRow);
+    newConfig.mediaBrowserItemsPerRow = isNaN(itemsPerRow) ? 4 : itemsPerRow;
     this.config = newConfig;
   }
 

diff --git a/src/sections/media-browser.ts b/src/sections/media-browser.ts
@@ -33,8 +33,8 @@ export class MediaBrowser extends LitElement {
       until(
         this.getFavorites(this.activePlayer).then((items) => {
           if (items?.length) {
-            const showFavoritesAsIcons = (this.config.mediaBrowserItemsPerRow ?? 4) > 1;
-            if (showFavoritesAsIcons) {
+            const itemsPerRow = this.config.mediaBrowserItemsPerRow || 4;
+            if (itemsPerRow > 1) {
               return html`
                 <sonos-media-browser-icons
                   .items=${items}