From 837f5dff0948cfe6d3dbc5a4fc082204d61383a5 Mon Sep 17 00:00:00 2001
From: Carl Schwan <carl@carlschwan.eu>
Date: Thu, 3 Dec 2020 16:44:04 +0100
Subject: [PATCH] Fix delegate not all having the same height in the room list

Fix: #112
---
 imports/NeoChat/Page/RoomListPage.qml | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/imports/NeoChat/Page/RoomListPage.qml b/imports/NeoChat/Page/RoomListPage.qml
index 4aceca23e..31056adb3 100644
--- a/imports/NeoChat/Page/RoomListPage.qml
+++ b/imports/NeoChat/Page/RoomListPage.qml
@@ -95,7 +95,7 @@ Kirigami.ScrollablePage {
             id: roomListItem
             property bool itemVisible: model.categoryVisible || sortFilterRoomListModel.filterText.length > 0 || Config.mergeRoomList
             visible: itemVisible
-            height: itemVisible ? roomLayout.implicitHeight : 0
+            height: itemVisible ? roomLayout.height : 0
             highlighted: roomManager.currentRoom && roomManager.currentRoom.name === name
             focus: true
             action: Kirigami.Action {
@@ -114,10 +114,12 @@ Kirigami.ScrollablePage {
             contentItem: RowLayout {
                 id: roomLayout
                 spacing: Kirigami.Units.largeSpacing
+                height: Kirigami.Units.gridUnit * 3
+                implicitHeight: Kirigami.Units.gridUnit * 3
 
                 Kirigami.Avatar {
                     id: roomAvatar
-                    property int size: parent.height - Kirigami.Units.smallSpacing * 2;
+                    property int size: Kirigami.Units.gridUnit * 2
                     Layout.minimumHeight: size
                     Layout.maximumHeight: size
                     Layout.minimumWidth: size