Skip to content

Optimized threads conversation view. #62

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
akwotom wants to merge 3 commits into
base: master
Choose a base branch
from
+150 −82
Open

Optimized threads conversation view. #62

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
829c195
Optimized threads conversation view, by caching frequently computed d…
akwotom Jul 10, 2026
f9cc617
Removed unnecessary old code.
akwotom Jul 10, 2026
c24d35d
Improved memory efficiency.
akwotom Jul 10, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion ...mms_android/src/main/java/com/afkanerd/smswithoutborders_libsmsmms/data/dao/ThreadsDao.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ import com.afkanerd.smswithoutborders_libsmsmms.data.entities.Threads
@Dao
interface ThreadsDao {

@Query("SELECT * FROM Threads WHERE isArchive = 0 AND address IS NOT NULL ORDER BY isPinned DESC, date DESC")
@Query("SELECT * FROM Threads WHERE isArchive = 0 AND isBlocked = 0 AND address IS NOT NULL ORDER BY isPinned DESC, date DESC")
fun getThreads0(): PagingSource<Int, Threads>

@Query("SELECT * FROM Threads WHERE isArchive = 0 AND address IS NOT NULL AND isPinned = 1 ORDER BY date DESC, isPinned DESC")
Expand Down
2 changes: 2 additions & 0 deletions ...s_android/src/main/java/com/afkanerd/smswithoutborders_libsmsmms/data/entities/Threads.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,13 @@
package com.afkanerd.smswithoutborders_libsmsmms.data.entities

import android.provider.Telephony
import androidx.compose.runtime.Immutable
import androidx.room.ColumnInfo
import androidx.room.Entity
import androidx.room.Index
import androidx.room.PrimaryKey

@Immutable

@sherlockwisdom sherlockwisdom Jul 10, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this for?

@akwotom akwotom Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This allows jetpack to optimize areas that depend on the annotated data structure, by explicitly stating that the underlying fields will not change.

@Entity(indices = [Index(value = ["address"], unique = true)])
data class Threads(
@PrimaryKey var threadId: Int,
Expand Down
64 changes: 25 additions & 39 deletions ...roid/src/main/java/com/afkanerd/smswithoutborders_libsmsmms/ui/ThreadsConversationMain.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
package com.afkanerd.smswithoutborders_libsmsmms.ui

import android.content.Context
import android.os.Debug
import android.provider.BlockedNumberContract.isBlocked
import android.provider.Telephony
import androidx.activity.compose.BackHandler
import androidx.compose.animation.core.Animatable
Expand Down Expand Up @@ -33,12 +33,10 @@ import androidx.compose.material.icons.filled.Menu
import androidx.compose.material.icons.filled.MoreVert
import androidx.compose.material.icons.filled.Notifications
import androidx.compose.material.icons.filled.NotificationsOff
import androidx.compose.material.icons.filled.PianoOff
import androidx.compose.material.icons.filled.PushPin
import androidx.compose.material.icons.filled.Search
import androidx.compose.material.icons.filled.Unarchive
import androidx.compose.material.icons.outlined.Block
import androidx.compose.material.icons.outlined.PushPin
import androidx.compose.material.icons.outlined.Remove
import androidx.compose.material.icons.rounded.Delete
import androidx.compose.material3.CenterAlignedTopAppBar
Expand All @@ -57,6 +55,7 @@ import androidx.compose.material3.TopAppBarDefaults
import androidx.compose.material3.rememberTopAppBarState
import androidx.compose.runtime.Composable
import androidx.compose.runtime.DisposableEffect
import androidx.compose.runtime.Immutable
import androidx.compose.runtime.LaunchedEffect
import androidx.compose.runtime.collectAsState
import androidx.compose.runtime.getValue
Expand Down Expand Up @@ -85,12 +84,8 @@ import androidx.lifecycle.viewmodel.compose.viewModel
import androidx.navigation.NavController
import androidx.navigation.compose.rememberNavController
import androidx.paging.LoadState.Loading
import androidx.paging.PagingData
import androidx.paging.compose.LazyPagingItems
import androidx.paging.compose.collectAsLazyPagingItems
import androidx.paging.compose.itemKey
import coil3.Uri
import coil3.toUri
import com.afkanerd.lib_smsmms_android.R
import com.afkanerd.smswithoutborders_libsmsmms.data.data.models.DateTimeUtils
import com.afkanerd.smswithoutborders_libsmsmms.data.entities.Threads
Expand All @@ -114,8 +109,8 @@ import com.afkanerd.smswithoutborders_libsmsmms.ui.viewModels.ThreadsViewModel
import com.google.accompanist.permissions.ExperimentalPermissionsApi
import com.google.accompanist.permissions.isGranted
import com.google.accompanist.permissions.rememberPermissionState
import kotlinx.coroutines.flow.flowOf
import kotlinx.coroutines.launch
import kotlin.collections.map
import kotlin.math.roundToInt

data class ThreadsConversationParameters(
Expand Down Expand Up @@ -151,7 +146,7 @@ fun ThreadConversationLayout(
val messagesAreLoading = threadsViewModel.messagesLoading
val secondaryMessagesAreLoading = threadsViewModel.secondaryMessagesLoading

var inboxType by remember { mutableStateOf(ThreadsViewModel.InboxType.INBOX )}
var inboxType by remember { mutableStateOf(ThreadsViewModel.InboxType.INBOX) }
val isAndroidJUnitTest = try {
Class.forName("androidx.test.runner.AndroidJUnitRunner")
true
Expand All @@ -173,11 +168,15 @@ fun ThreadConversationLayout(

val selectedItems by threadsViewModel.selectedItems.collectAsState()

val inboxMessagesPagers = threadsViewModel.getThreads(context)
val archivedMessagesPagers = threadsViewModel.getArchives(context)

val inboxMessagesPagers =
threadsViewModel.getThreads(context)
val archivedMessagesPagers =
threadsViewModel.getArchives(context)
val draftMessagesPagers = threadsViewModel.getDrafts(context)
val mutedMessagesPagers = threadsViewModel.getIsMute(context)
val blockedMessagesPager = threadsViewModel.getIsBlocked(context)
val blockedMessagesPager =
threadsViewModel.getIsBlocked(context)

val inboxMessagesItems = inboxMessagesPagers.collectAsLazyPagingItems()
val archivedMessagesItems = archivedMessagesPagers.collectAsLazyPagingItems()
Expand All @@ -204,6 +203,8 @@ fun ThreadConversationLayout(

val scope = rememberCoroutineScope()



BackHandler(
inboxType != ThreadsViewModel.InboxType.INBOX ||
!selectedItems.isEmpty()
Expand Down Expand Up @@ -563,12 +564,13 @@ fun ThreadConversationLayout(
if (!isDefault || !readPhoneStatePermission.status.isGranted) {
DefaultCheckMain { isDefault = context.isDefault() }
}
if(secondaryMessagesAreLoading || isAndroidJUnitTest || inPreviewMode)
if (secondaryMessagesAreLoading || isAndroidJUnitTest || inPreviewMode)
LinearProgressIndicator(
Modifier.fillMaxWidth()
Modifier
.fillMaxWidth()
.testTag("secondaryMessagesAreLoading")
)
if(messagesAreLoading || inPreviewMode) {
if (messagesAreLoading || inPreviewMode) {
Column(
modifier = Modifier.fillMaxSize(),
verticalArrangement = Arrangement.Center,
Expand Down Expand Up @@ -625,9 +627,10 @@ fun ThreadConversationLayout(
) {
items(
count = displayedInbox.itemCount,
key = displayedInbox.itemKey { it.threadId }
key = displayedInbox.itemKey { it.raw.threadId }
) { index ->
val thread = displayedInbox[index] ?: return@items
val enriched = displayedInbox[index] ?: return@items
val thread = enriched.raw;

val offsetX = remember { Animatable(0f) }
val threshold = 300f
Expand Down Expand Up @@ -700,24 +703,6 @@ fun ThreadConversationLayout(
) {
val address = thread.address

val isBlocked = remember(thread.threadId) {
if (isDefault)
threadsViewModel.isBlocked(
context, thread,
blockedMessagesItems.itemSnapshotList.items
)
else false
}

val contactName = remember(address) {
if (isDefault)
context.retrieveContactName(address)
else address
}

val contactPhotoUri by threadsViewModel
.contactPhoto(context, address)
.collectAsState()

val isSelected = remember(selectedItems) {
selectedItems.contains(thread)
Expand All @@ -732,12 +717,12 @@ fun ThreadConversationLayout(

ThreadConversationCard(
id = thread.threadId,
name = contactName ?: address,
name = enriched.contactName ?: thread.address,
content = thread.snippet,
date = date,
isRead = !thread.unread,
isContact = isDefault && !contactName.isNullOrBlank(),
isBlocked = isBlocked,
isContact = enriched.isContact,
isBlocked = inboxType == ThreadsViewModel.InboxType.BLOCKED,
isPinned = thread.isPinned,
modifier = Modifier.combinedClickable(
onClick = {
Expand Down Expand Up @@ -784,7 +769,7 @@ fun ThreadConversationLayout(
type = thread.type,
unreadCount = thread.unreadCount,
mms = thread.isMms,
contactPhotoUri = contactPhotoUri,
contactPhotoUri = enriched.contactPhotoUri,
)
}
}
Expand Down Expand Up @@ -816,6 +801,7 @@ fun ThreadConversationLayout(

}


@Preview
@Composable
fun PreviewMessageCard() {
Expand Down
Loading