remove param from web pref dialog onDismiss call

This commit is contained in:
Satindar Dhillon 2022-10-12 08:04:14 -07:00
parent 84de7ea640
commit 41e5d3000b
2 changed files with 6 additions and 12 deletions

View file

@ -9,19 +9,19 @@ import androidx.compose.ui.unit.dp
import androidx.compose.ui.window.Dialog
@Composable
fun WebPreferencesDialog(onDismiss: (WebPreferences?) -> Unit) {
Dialog(onDismissRequest = { onDismiss(null) }) {
fun WebPreferencesDialog(onDismiss: () -> Unit) {
Dialog(onDismissRequest = { onDismiss() }) {
Surface(
shape = RoundedCornerShape(16.dp),
color = Color.White
) {
WebPreferencesView(onDismiss)
WebPreferencesView()
}
}
}
@Composable
fun WebPreferencesView(onDismiss: (WebPreferences?) -> Unit) {
fun WebPreferencesView() {
Text("Web Prefs")
}

View file

@ -44,7 +44,6 @@ fun WebReaderLoadingContainer(slug: String, webReaderViewModel: WebReaderViewMod
)
var showWebPreferencesDialog by remember { mutableStateOf(false ) }
var webPreferences by remember { mutableStateOf(defaultWebPreferences ) }
val webReaderParams: WebReaderParams? by webReaderViewModel.webReaderParamsLiveData.observeAsState(null)
val annotation: String? by webReaderViewModel.annotationLiveData.observeAsState(null)
@ -87,7 +86,7 @@ fun WebReaderLoadingContainer(slug: String, webReaderViewModel: WebReaderViewMod
.requiredHeight(height = maxToolbarHeight)
) {
}
WebReader(webReaderParams!!, webPreferences, webReaderViewModel)
WebReader(webReaderParams!!, defaultWebPreferences, webReaderViewModel)
}
TopAppBar(
@ -108,12 +107,7 @@ fun WebReaderLoadingContainer(slug: String, webReaderViewModel: WebReaderViewMod
)
if (showWebPreferencesDialog) {
WebPreferencesDialog { preferences ->
if (preferences != null) {
webPreferences = preferences!!
showWebPreferencesDialog = false
}
WebPreferencesDialog {
showWebPreferencesDialog = false
}
}