From 5545f3751aaf989770dd6b67b518e81c368444fd Mon Sep 17 00:00:00 2001
From: Jake Roseman <122034773+jakeroseman@users.noreply.github.com>
Date: Mon, 14 Oct 2024 12:21:49 +0100
Subject: [PATCH] Remove LaunchedEffect workaround (#379)

* Remove LaunchedEffect workaround for library bug

* Apply Spotless

* Changed single var to val

---------

Co-authored-by: jakeroseman <jakeroseman@users.noreply.github.com>
---
 .../compose/snippets/components/Tooltips.kt   | 35 +++----------------
 1 file changed, 4 insertions(+), 31 deletions(-)

diff --git a/compose/snippets/src/main/java/com/example/compose/snippets/components/Tooltips.kt b/compose/snippets/src/main/java/com/example/compose/snippets/components/Tooltips.kt
index e43ac2d9..1b5d9ea0 100644
--- a/compose/snippets/src/main/java/com/example/compose/snippets/components/Tooltips.kt
+++ b/compose/snippets/src/main/java/com/example/compose/snippets/components/Tooltips.kt
@@ -34,12 +34,9 @@ import androidx.compose.material3.Text
 import androidx.compose.material3.TextButton
 import androidx.compose.material3.TooltipBox
 import androidx.compose.material3.TooltipDefaults
-import androidx.compose.material3.TooltipState
+import androidx.compose.material3.rememberTooltipState
 import androidx.compose.runtime.Composable
-import androidx.compose.runtime.LaunchedEffect
 import androidx.compose.runtime.getValue
-import androidx.compose.runtime.mutableStateOf
-import androidx.compose.runtime.remember
 import androidx.compose.runtime.setValue
 import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
@@ -79,14 +76,13 @@ fun PlainTooltipExample(
     modifier: Modifier = Modifier,
     plainTooltipText: String = "Add to favorites"
 ) {
-    var tooltipState by remember { mutableStateOf(TooltipState()) }
     TooltipBox(
         modifier = modifier,
         positionProvider = TooltipDefaults.rememberPlainTooltipPositionProvider(),
         tooltip = {
             PlainTooltip { Text(plainTooltipText) }
         },
-        state = tooltipState
+        state = rememberTooltipState()
     ) {
         IconButton(onClick = { /* Do something... */ }) {
             Icon(
@@ -95,13 +91,6 @@ fun PlainTooltipExample(
             )
         }
     }
-
-    // Reset tooltipState after closing the tooltip.
-    LaunchedEffect(tooltipState.isVisible) {
-        if (!tooltipState.isVisible) {
-            tooltipState = TooltipState()
-        }
-    }
 }
 
 // [END android_compose_components_plaintooltipexample]
@@ -120,8 +109,6 @@ fun RichTooltipExample(
     richTooltipSubheadText: String = "Rich Tooltip",
     richTooltipText: String = "Rich tooltips support multiple lines of informational text."
 ) {
-    var tooltipState by remember { mutableStateOf(TooltipState(isPersistent = true)) }
-
     TooltipBox(
         modifier = modifier,
         positionProvider = TooltipDefaults.rememberRichTooltipPositionProvider(),
@@ -132,7 +119,7 @@ fun RichTooltipExample(
                 Text(richTooltipText)
             }
         },
-        state = tooltipState
+        state = rememberTooltipState()
     ) {
         IconButton(onClick = { /* Icon button's click event */ }) {
             Icon(
@@ -141,13 +128,6 @@ fun RichTooltipExample(
             )
         }
     }
-
-    // Reset tooltipState after closing the tooltip.
-    LaunchedEffect(tooltipState.isVisible) {
-        if (!tooltipState.isVisible) {
-            tooltipState = TooltipState(isPersistent = true)
-        }
-    }
 }
 // [END android_compose_components_richtooltipexample]
 
@@ -166,7 +146,7 @@ fun AdvancedRichTooltipExample(
     richTooltipText: String = "Rich tooltips support multiple lines of informational text.",
     richTooltipActionText: String = "Dismiss"
 ) {
-    var tooltipState by remember { mutableStateOf(TooltipState(isPersistent = true)) }
+    val tooltipState = rememberTooltipState()
 
     TooltipBox(
         modifier = modifier,
@@ -195,13 +175,6 @@ fun AdvancedRichTooltipExample(
             )
         }
     }
-
-    // Reset tooltipState after closing the tooltip.
-    LaunchedEffect(tooltipState.isVisible) {
-        if (!tooltipState.isVisible) {
-            tooltipState = TooltipState(isPersistent = true)
-        }
-    }
 }
 // [END android_compose_components_advancedrichtooltipexample]