Merge pull request #202 from vector-im/fix/mna/trailing-icon-search-bar

Fix display of trailing icon in the search bar

changelog.d/202.wip

@@ -0,0 +1 @@
+[Create and join rooms] Fix display of trailing icon in the search bar

features/createroom/impl/src/main/kotlin/io/element/android/features/createroom/impl/root/CreateRoomRootView.kt

@@ -154,18 +154,24 @@ fun CreateRoomSearchBar(
         leadingIcon = if (active) {
             { BackButton(onClick = { onActiveChanged(false) }) }
         } else null,
-        trailingIcon = {
+        trailingIcon = when {
-            if (active) {
+            active && text.isNotEmpty() -> {
-                IconButton(onClick = { onTextChanged("") }) {
+                {
-                    Icon(Icons.Default.Close, stringResource(StringR.string.a11y_clear))
+                    IconButton(onClick = { onTextChanged("") }) {
+                        Icon(Icons.Default.Close, stringResource(StringR.string.a11y_clear))
+                    }
+                }
+            }
+            !active -> {
+                {
+                    Icon(
+                        imageVector = Icons.Default.Search,
+                        contentDescription = stringResource(StringR.string.search),
+                        modifier = Modifier.alpha(0.4f), // FIXME align on Design system theme (removing alpha should be fine)
+                    )
                 }
-            } else {
-                Icon(
-                    imageVector = Icons.Default.Search,
-                    contentDescription = stringResource(StringR.string.search),
-                    modifier = Modifier.alpha(0.4f), // FIXME align on Design system theme (removing alpha should be fine)
-                )
             }
+            else -> null
         },
         shape = if (!active) SearchBarDefaults.dockedShape else SearchBarDefaults.fullScreenShape,
         colors = if (!active) SearchBarDefaults.colors() else SearchBarDefaults.colors(containerColor = Color.Transparent),