Sulkta-OSS/element-x-ada
3
0
Fork 0
Code Issues 1 Pull requests 10 Projects Releases Packages Wiki Activity Actions

Merge pull request #2353 from element-hq/feature/bma/removeDuplicate

Browse source 
Do not use a key for the LazyColumn.
This commit is contained in:
Benoit Marty 2024-02-07 09:47:02 +01:00 • committed by GitHub
commit 61a449ab25
1 changed files with 2 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
features/roomlist/impl/src/main/kotlin/io/element/android/features/roomlist/impl/RoomListView.kt
View file

@ -259,10 +259,11 @@ private fun RoomListContent(
} }
val roomList = state.roomList.dataOrNull().orEmpty() val roomList = state.roomList.dataOrNull().orEmpty()
// Note: do not use a key for the LazyColumn, or the scroll will not behave as expected if a room
// is moved to the top of the list.
itemsIndexed( itemsIndexed(
items = roomList, items = roomList,
contentType = { _, room -> room.contentType() }, contentType = { _, room -> room.contentType() },
key = { _, room -> room.roomId.value }
) { index, room -> ) { index, room ->
RoomSummaryRow( RoomSummaryRow(
room = room, room = room,