From 278091e6efce09f03262779bfcb5ce052665acad Mon Sep 17 00:00:00 2001
From: Vitor Pamplona <vitor@vitorpamplona.com>
Date: Mon, 22 May 2023 17:51:37 -0400
Subject: [PATCH] Moves default state of url previews to the remember function

---
 .../amethyst/ui/components/UrlPreview.kt      | 19 +++++++++++--------
 1 file changed, 11 insertions(+), 8 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 14fb7bd9f..b4d914a40 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,16 +19,19 @@ import kotlinx.coroutines.launch
 
 @Composable
 fun UrlPreview(url: String, urlText: String) {
-    val default = UrlCachedPreviewer.cache[url]?.let {
-        if (it.allFetchComplete() && it.url == url) {
-            UrlPreviewState.Loaded(it)
-        } else {
-            UrlPreviewState.Empty
-        }
-    } ?: UrlPreviewState.Loading
     val context = LocalContext.current
 
-    var urlPreviewState by remember { mutableStateOf<UrlPreviewState>(default) }
+    var urlPreviewState by remember {
+        val default = UrlCachedPreviewer.cache[url]?.let {
+            if (it.allFetchComplete() && it.url == url) {
+                UrlPreviewState.Loaded(it)
+            } else {
+                UrlPreviewState.Empty
+            }
+        } ?: UrlPreviewState.Loading
+
+        mutableStateOf(default)
+    }
     val scope = rememberCoroutineScope()
 
     // Doesn't use a viewModel because of viewModel reusing issues (too many UrlPreview are created).