From bab314ebc96698175663d41d9e44d19dc661b194 Mon Sep 17 00:00:00 2001 From: Vitor Pamplona Date: Thu, 2 Mar 2023 13:38:05 -0500 Subject: [PATCH] Fixing Preview Error when the url is invalid --- .../vitorpamplona/amethyst/ui/components/UrlPreview.kt | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/app/src/main/java/com/vitorpamplona/amethyst/ui/components/UrlPreview.kt b/app/src/main/java/com/vitorpamplona/amethyst/ui/components/UrlPreview.kt index bd993f8725..d4adfc34d6 100644 --- a/app/src/main/java/com/vitorpamplona/amethyst/ui/components/UrlPreview.kt +++ b/app/src/main/java/com/vitorpamplona/amethyst/ui/components/UrlPreview.kt @@ -19,10 +19,16 @@ import kotlinx.coroutines.withContext @Composable fun UrlPreview(url: String, urlText: String) { - val default = UrlCachedPreviewer.cache[url]?.let { UrlPreviewState.Loaded(it) } ?: UrlPreviewState.Loading + val default = UrlCachedPreviewer.cache[url]?.let { + if (it.url == url) + UrlPreviewState.Loaded(it) + else + UrlPreviewState.Empty + + } ?: UrlPreviewState.Loading var context = LocalContext.current - var urlPreviewState by remember { mutableStateOf(default) } + var urlPreviewState by remember { mutableStateOf(UrlPreviewState.Loading) } // Doesn't use a viewModel because of viewModel reusing issues (too many UrlPreview are created). LaunchedEffect(url) {