nick/amethyst
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

Removes the use of by on the quick action menu's state because it was triggering recompositions

Browse Source
This commit is contained in:
Vitor Pamplona
2023-06-13 16:30:37 -04:00
parent 305ecc6a2e
commit 1cd39e0d8f
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/src/main/java/com/vitorpamplona/amethyst/ui/note/NoteQuickActionMenu.kt
+3 -3
View File
@@ -112,12 +112,12 @@ private fun VerticalDivider(color: Color) =
@Composable @Composable
fun LongPressToQuickAction(baseNote: Note, accountViewModel: AccountViewModel, content: @Composable (() -> Unit) -> Unit) { fun LongPressToQuickAction(baseNote: Note, accountViewModel: AccountViewModel, content: @Composable (() -> Unit) -> Unit) {
var popupExpanded by remember { mutableStateOf(false) } var popupExpanded = remember { mutableStateOf(false) }
val showPopup = remember { { popupExpanded = true } } val showPopup = remember(popupExpanded) { { popupExpanded.value = true } }
content(showPopup) content(showPopup)
NoteQuickActionMenu(baseNote, popupExpanded, { popupExpanded = false }, accountViewModel) NoteQuickActionMenu(baseNote, popupExpanded.value, { popupExpanded.value = false }, accountViewModel)
} }
@Composable @Composable