zapstore-mirror/com.darkwisp.app
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #491 from barrydeen/fix/notifications-freeze-indexof

Browse Source 
perf: avoid O(N^2) indexOf in notifications list
This commit is contained in:
Barry Deen
2026-04-23 10:30:19 -04:00
committed by GitHub
parent f4c716551b 503c46b256
commit b609fc56c0
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/src/main/kotlin/com/wisp/app/ui/screen/NotificationsScreen.kt
+3 -2
View File
@@ -22,6 +22,7 @@ import androidx.compose.foundation.layout.widthIn
import androidx.compose.foundation.layout.wrapContentSize
import androidx.compose.foundation.lazy.LazyColumn
import androidx.compose.foundation.lazy.items
import androidx.compose.foundation.lazy.itemsIndexed
import androidx.compose.foundation.lazy.rememberLazyListState
import androidx.compose.foundation.shape.CircleShape
import androidx.compose.foundation.shape.RoundedCornerShape
@@ -381,9 +382,9 @@ fun NotificationsScreen(
}
)
}
items(items = notifications, key = { it.id }, contentType = { "notification" }) { item ->
itemsIndexed(items = notifications, key = { _, it -> it.id }, contentType = { _, _ -> "notification" }) { index, item ->
val isExpanded = expandedId == item.id
val itemIndex = notifications.indexOf(item) + 1 // +1 for summary header
val itemIndex = index + 1 // +1 for summary header
val coroutineScope = rememberCoroutineScope()
ZenNotificationRow(
item = item,