From: Scott Murray <scott.murray@konsulko.com>
Date: Sat, 18 Jun 2022 01:31:45 +0000 (-0400)
Subject: Fix layout issues
X-Git-Tag: 13.93.0^0
X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?p=apps%2Fmediaplayer.git;a=commitdiff_plain;h=0aefba20fe07dbb79d7b3683ab38c0b0b313a655

Fix layout issues

Fixes:
- Tweak the album art image anchoring and scaling so it is not covered
  by the control bar.
- Tweak some item preferred heights to unify the control bar position
  with the radio app at a position closer to mid-screen.

Bug-AGL: SPEC-4444

Signed-off-by: Scott Murray <scott.murray@konsulko.com>
Change-Id: I641113ff50df1242407a5dd95ce8364b5a49a35c
(cherry picked from commit 5461945a951debb86430b4e8c94e5666e543ba22)
---

diff --git a/app/MediaPlayer.qml b/app/MediaPlayer.qml
index 03c85c4..fd4db58 100644
--- a/app/MediaPlayer.qml
+++ b/app/MediaPlayer.qml
@@ -140,17 +140,19 @@ ApplicationWindow {
             Layout.fillHeight: true
             Layout.preferredHeight: 1080
             clip: true
+
             Image {
+                anchors.top: parent.top
                 anchors.left: parent.left
                 anchors.right: parent.right
-                anchors.bottom: parent.bottom
-                height: sourceSize.height * width / sourceSize.width
-                fillMode: Image.PreserveAspectCrop
+                anchors.bottom: controls.top
+                fillMode: Image.PreserveAspectFit
                 source: AlbumArt
                 visible: player.av_connected === false
             }
 
             Item {
+                id: controls
                 anchors.left: parent.left
                 anchors.right: parent.right
                 anchors.bottom: parent.bottom
@@ -292,7 +294,7 @@ ApplicationWindow {
         Item {
             Layout.fillWidth: true
             Layout.fillHeight: true
-            Layout.preferredHeight: 407
+            Layout.preferredHeight: 480
 
             ListView {
                 anchors.fill: parent