From c0e2041efe4d69c9c418b8fa7fd895c4f4e0b800 Mon Sep 17 00:00:00 2001 From: tarsin Date: Wed, 2 Aug 2023 10:25:09 +0800 Subject: [PATCH] Cleanup code --- .../java/com/hippo/ehviewer/ui/tools/Dialog.kt | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/app/src/main/java/com/hippo/ehviewer/ui/tools/Dialog.kt b/app/src/main/java/com/hippo/ehviewer/ui/tools/Dialog.kt index 18f559775..39e224e37 100644 --- a/app/src/main/java/com/hippo/ehviewer/ui/tools/Dialog.kt +++ b/app/src/main/java/com/hippo/ehviewer/ui/tools/Dialog.kt @@ -71,7 +71,7 @@ class DialogState { content = null } - suspend inline fun dialog(crossinline block: @Composable (CancellableContinuation) -> Unit) = suspendCancellableCoroutine { cont -> + suspend inline fun dialog(crossinline block: @Composable (CancellableContinuation) -> Unit) = suspendCancellableCoroutine { cont -> cont.invokeOnCancellation { dismiss() } val realContinuation = object : CancellableContinuation by cont { override fun resumeWith(result: Result) { @@ -93,7 +93,7 @@ class DialogState { Text(text = stringResource(id = android.R.string.ok)) } }, - title = title?.let { { Text(text = stringResource(id = title)) } }, + title = title.ifNotNullThen { Text(text = stringResource(id = title!!)) }, text = { block(impl) }, ) } @@ -119,7 +119,7 @@ class DialogState { Text(text = stringResource(id = android.R.string.ok)) } }, - title = title?.let { { Text(text = title) } }, + title = title.ifNotNullThen { Text(text = title!!) }, text = { OutlinedTextField( value = state, @@ -158,14 +158,12 @@ class DialogState { Text(text = stringResource(id = confirmText ?: android.R.string.ok)) } }, - dismissButton = dismissText?.let { - { - TextButton(onClick = { cont.cancel() }) { - Text(text = stringResource(id = dismissText)) - } + dismissButton = dismissText.ifNotNullThen { + TextButton(onClick = { cont.cancel() }) { + Text(text = stringResource(id = dismissText!!)) } }, - title = title?.let { { Text(text = stringResource(id = title)) } }, + title = title.ifNotNullThen { Text(text = stringResource(id = title!!)) }, text = text, ) }