nik/bitchat-android
1
0
Fork 0
mirror of https://github.com/permissionlesstech/bitchat-android.git synced 2026-08-15 06:56:30 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge 750abafaadb9422c3245e89cde9e4f5212c70e4f into 094657efa0aabbb6f71c9050149d1d01aee96400

Browse Source
This commit is contained in:
heyaim 2026-08-02 22:05:49 -05:00 committed by GitHub
commit 10fb904609
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 57 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
app/src/main/java/com/bitchat/android/ui/ChatScreen.kt
View File

@ -100,11 +100,14 @@ fun ChatScreen(viewModel: ChatViewModel) {
var isScrolledUp by remember { mutableStateOf(false) }
LaunchedEffect(selectedPrivatePeer) {
messageText = TextFieldValue(
selectedPrivatePeer
?.let(viewModel::conversationDraft)
.orEmpty()
)
val draft = selectedPrivatePeer
?.let(viewModel::conversationDraft)
.orEmpty()
messageText = TextFieldValue(draft)
// Setting the field in code does not run onMessageTextChange, so the
// popups have to be re-synced with the restored draft here.
viewModel.updateCommandSuggestions(draft)
viewModel.updateMentionSuggestions(draft)
}
// Show password dialog when needed
@ -342,6 +345,11 @@ fun ChatScreen(viewModel: ChatViewModel) {
text = newText,
selection = TextRange(newText.length)
)
// Setting the field in code does not run onMessageTextChange, so
// the popups have to be re-synced with the new text here. The
// inserted mention ends with a space, which hides an open popup.
viewModel.updateCommandSuggestions(newText)
viewModel.updateMentionSuggestions(newText)
},
onMessageLongPress = { message ->
// Message long press - open user action sheet with message context

44
app/src/test/java/com/bitchat/android/ui/CommandProcessorTest.kt
View File

@ -188,4 +188,48 @@ class CommandProcessorTest() {
assertFalse(chatState.getShowMentionSuggestionsValue())
assertTrue(chatState.getMentionSuggestionsValue().isEmpty())
}
@Test
fun `text set to an inserted mention hides both popups`() {
// A command popup is open while the user taps a nickname in the message list.
commandProcessor.updateCommandSuggestions("/")
assertTrue(chatState.getShowCommandSuggestionsValue())
// The composer re-syncs both popups with the inserted text, which ends in a
// space, so neither popup matches it.
commandProcessor.updateCommandSuggestions("/ @alice ")
commandProcessor.updateMentionSuggestions("/ @alice ", meshService, viewModel = null)
assertFalse(chatState.getShowCommandSuggestionsValue())
assertTrue(chatState.getCommandSuggestionsValue().isEmpty())
assertFalse(chatState.getShowMentionSuggestionsValue())
assertTrue(chatState.getMentionSuggestionsValue().isEmpty())
}
@Test
fun `a mention followed by a space no longer matches the mention popup`() {
whenever(meshService.myPeerID).thenReturn("me")
whenever(meshService.getPeerNicknames()).thenReturn(mapOf("peer-1" to "alice"))
commandProcessor.updateMentionSuggestions("@a", meshService, viewModel = null)
assertTrue(chatState.getShowMentionSuggestionsValue())
commandProcessor.updateMentionSuggestions("@alice ", meshService, viewModel = null)
assertFalse(chatState.getShowMentionSuggestionsValue())
assertTrue(chatState.getMentionSuggestionsValue().isEmpty())
}
@Test
fun `a restored command draft reopens the command popup`() {
// Switching conversations restores the draft in code; re-syncing with the
// restored text brings the popup back for a command draft.
commandProcessor.updateCommandSuggestions("/j")
assertTrue(chatState.getShowCommandSuggestionsValue())
assertTrue(chatState.getCommandSuggestionsValue().isNotEmpty())
// An empty restored draft leaves both popups hidden.
commandProcessor.updateCommandSuggestions("")
commandProcessor.updateMentionSuggestions("", meshService, viewModel = null)
assertFalse(chatState.getShowCommandSuggestionsValue())
assertFalse(chatState.getShowMentionSuggestionsValue())
}
}