Merge pull request jellyfin#1678 from jellyfin/banner

Fix banner height and use primary image as fallback
iwalton3 · Jul 30, 2020 · 34d0b67 · 34d0b67
2 parents 5d6b0b2 + 24cd5e1
commit 34d0b67
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/assets/css/librarybrowser.css b/src/assets/css/librarybrowser.css
@@ -646,7 +646,7 @@
 .layout-desktop .detailRibbon,
 .layout-tv .detailRibbon {
     margin-top: -7.2em;
-    height: 7.18em;
+    height: 7.2em;
 }
 
 .layout-desktop .noBackdrop .detailRibbon,

diff --git a/src/controllers/itemDetails/index.js b/src/controllers/itemDetails/index.js
@@ -523,6 +523,14 @@ define(['loading', 'appRouter', 'layoutManager', 'connectionManager', 'userSetti
             });
             imageLoader.lazyImage(itemBackdropElement, imgUrl);
             hasbackdrop = true;
+        } else if (item.ImageTags && item.ImageTags.Primary) {
+            imgUrl = apiClient.getScaledImageUrl(item.Id, {
+                type: 'Primary',
+                maxWidth: dom.getScreenWidth(),
+                tag: item.ImageTags.Primary
+            });
+            imageLoader.lazyImage(itemBackdropElement, imgUrl);
+            hasbackdrop = true;
         } else {
             itemBackdropElement.style.backgroundImage = '';
         }