From d7d55fa4f910715b5963c217e92e07cb18152e7d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sebastian=20K=C3=BCrten?= <sebastian@mobanisto.de>
Date: Tue, 14 Sep 2021 21:02:40 +0200
Subject: [PATCH] Clarify why we need to reverse the message items for layout
---
.../briar/desktop/paul/views/PrivateMessageView.kt | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/main/kotlin/org/briarproject/briar/desktop/paul/views/PrivateMessageView.kt b/src/main/kotlin/org/briarproject/briar/desktop/paul/views/PrivateMessageView.kt
index ded78f2244..a0626f2e57 100644
--- a/src/main/kotlin/org/briarproject/briar/desktop/paul/views/PrivateMessageView.kt
+++ b/src/main/kotlin/org/briarproject/briar/desktop/paul/views/PrivateMessageView.kt
@@ -423,7 +423,8 @@ fun ChatState(
val visitor = ChatHistoryConversationVisitor(chat, messagingManager)
val messageHeaders: List<ConversationMessageHeader> = ArrayList(conversationManager.getMessageHeaders(id))
Collections.sort(messageHeaders, ConversationMessageHeaderComparator())
- // FIXME: for some reason messages are displayed in reverse order
+ // Reverse order here because we're using reverseLayout=true on the LazyColumn to display items
+ // from bottom to top
Collections.reverse(messageHeaders)
for (header in messageHeaders) {
header.accept(visitor)
--
GitLab