Set Android View ID on TreehouseLayout by default

If one doesn't explicitly set an Android View ID, `onSaveInstanceState` is never invoked, and thus `rememberSaveable` will never work as intended.
cashapp · Sep 28, 2023 · 7ec4832 · 7ec4832
1 parent 89750b1
commit 7ec4832
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/redwood-treehouse-host/src/androidMain/kotlin/app/cash/redwood/treehouse/TreehouseLayout.kt b/redwood-treehouse-host/src/androidMain/kotlin/app/cash/redwood/treehouse/TreehouseLayout.kt
@@ -57,6 +57,12 @@ public class TreehouseLayout(
 
   override var saveCallback: TreehouseView.SaveCallback? = null
 
+  init {
+    // The view needs to have an id for Android to populate saved data back
+    @SuppressLint("ResourceType")
+    id = 9000
+  }
+
   override fun onAttachedToWindow() {
     super.onAttachedToWindow()
     readyForContent = true

diff --git a/...iews/src/main/kotlin/com/example/redwood/emojisearch/android/views/EmojiSearchActivity.kt b/...iews/src/main/kotlin/com/example/redwood/emojisearch/android/views/EmojiSearchActivity.kt
@@ -73,8 +73,6 @@ class EmojiSearchActivity : ComponentActivity() {
 
     setContentView(
       TreehouseLayout(this, widgetSystem, onBackPressedDispatcher).apply {
-        // The view needs to have an id for Android to populate saved data back
-        this.id = 9000
         treehouseContentSource.bindWhenReady(this, treehouseApp)
       },
     )