Make Search retain the query and results (#1579)

chrisbanes · Oct 8, 2023 · 250dacd · 250dacd
1 parent 9639bb5
commit 250dacd
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/common/ui/compose/src/commonMain/kotlin/app/tivi/common/compose/ui/SearchTextField.kt b/common/ui/compose/src/commonMain/kotlin/app/tivi/common/compose/ui/SearchTextField.kt
@@ -11,8 +11,8 @@ import androidx.compose.material.icons.filled.Search
 import androidx.compose.material3.ExperimentalMaterial3Api
 import androidx.compose.material3.Icon
 import androidx.compose.material3.IconButton
+import androidx.compose.material3.OutlinedTextField
 import androidx.compose.material3.Text
-import androidx.compose.material3.TextField
 import androidx.compose.runtime.Composable
 import androidx.compose.ui.ExperimentalComposeUiApi
 import androidx.compose.ui.Modifier
@@ -33,7 +33,7 @@ fun SearchTextField(
 ) {
     val keyboardController = LocalSoftwareKeyboardController.current
 
-    TextField(
+    OutlinedTextField(
         value = value,
         onValueChange = onValueChange,
         leadingIcon = {

diff --git a/ui/search/src/commonMain/kotlin/app/tivi/home/search/SearchPresenter.kt b/ui/search/src/commonMain/kotlin/app/tivi/home/search/SearchPresenter.kt
@@ -19,6 +19,7 @@ import app.tivi.screens.SearchScreen
 import app.tivi.screens.ShowDetailsScreen
 import app.tivi.util.Logger
 import app.tivi.util.onException
+import com.slack.circuit.retained.rememberRetained
 import com.slack.circuit.runtime.CircuitContext
 import com.slack.circuit.runtime.Navigator
 import com.slack.circuit.runtime.presenter.Presenter
@@ -54,12 +55,13 @@ class SearchPresenter(
     override fun present(): SearchUiState {
         val scope = rememberCoroutineScope()
 
-        var query by remember { mutableStateOf("") }
+        var query by rememberRetained { mutableStateOf("") }
+        var results by rememberRetained { mutableStateOf(emptyList<TiviShow>()) }
+
         val uiMessageManager = remember { UiMessageManager() }
 
         val loading by searchShows.inProgress.collectAsState(false)
         val message by uiMessageManager.message.collectAsState(null)
-        var results by remember { mutableStateOf(emptyList<TiviShow>()) }
 
         LaunchedEffect(query) {
             // delay for 300 milliseconds. This has the same effect as debounce