Browse Source

Fix `OidcView` preview

pull/3298/head
Jorge Martín 1 month ago
parent
commit
ae18efd2a9
  1. 4
      libraries/oidc/impl/src/main/kotlin/io/element/android/libraries/oidc/impl/webview/OidcView.kt

4
libraries/oidc/impl/src/main/kotlin/io/element/android/libraries/oidc/impl/webview/OidcView.kt

@ -27,6 +27,7 @@ import androidx.compose.runtime.mutableStateOf
import androidx.compose.runtime.remember import androidx.compose.runtime.remember
import androidx.compose.runtime.setValue import androidx.compose.runtime.setValue
import androidx.compose.ui.Modifier import androidx.compose.ui.Modifier
import androidx.compose.ui.platform.LocalInspectionMode
import androidx.compose.ui.tooling.preview.PreviewParameter import androidx.compose.ui.tooling.preview.PreviewParameter
import androidx.compose.ui.viewinterop.AndroidView import androidx.compose.ui.viewinterop.AndroidView
import io.element.android.libraries.core.bool.orFalse import io.element.android.libraries.core.bool.orFalse
@ -45,6 +46,7 @@ fun OidcView(
onNavigateBack: () -> Unit, onNavigateBack: () -> Unit,
modifier: Modifier = Modifier, modifier: Modifier = Modifier,
) { ) {
val isPreview = LocalInspectionMode.current
val oidcUrlParser = remember { OidcUrlParser() } val oidcUrlParser = remember { OidcUrlParser() }
var webView by remember { mutableStateOf<WebView?>(null) } var webView by remember { mutableStateOf<WebView?>(null) }
fun shouldOverrideUrl(url: String): Boolean { fun shouldOverrideUrl(url: String): Boolean {
@ -86,6 +88,7 @@ fun OidcView(
modifier = Modifier.padding(contentPadding), modifier = Modifier.padding(contentPadding),
factory = { context -> factory = { context ->
WebView(context).apply { WebView(context).apply {
if (!isPreview) {
webViewClient = oidcWebViewClient webViewClient = oidcWebViewClient
settings.apply { settings.apply {
@SuppressLint("SetJavaScriptEnabled") @SuppressLint("SetJavaScriptEnabled")
@ -96,6 +99,7 @@ fun OidcView(
domStorageEnabled = true domStorageEnabled = true
} }
loadUrl(state.oidcDetails.url) loadUrl(state.oidcDetails.url)
}
}.also { }.also {
webView = it webView = it
} }

Loading…
Cancel
Save