feature(space): introduce SpaceRoomVisibility and remove room count

features/home/impl/src/main/kotlin/io/element/android/features/home/impl/spaces/HomeSpacesView.kt

@@ -33,11 +33,7 @@ fun HomeSpacesView(
         when (space) {
             CurrentSpace.Root -> {
                 item {
-                    SpaceHeaderRootView(
+                    SpaceHeaderRootView(numberOfSpaces = state.spaceRooms.size)
-                        numberOfSpaces = state.spaceRooms.size,
-                        // TODO
-                        numberOfRooms = 0,
-                    )
                 }
             }
             is CurrentSpace.Space -> item {
@@ -45,10 +41,9 @@ fun HomeSpacesView(
                     avatarData = space.spaceRoom.getAvatarData(AvatarSize.SpaceHeader),
                     name = space.spaceRoom.name,
                     topic = space.spaceRoom.topic,
-                    joinRule = space.spaceRoom.joinRule,
+                    visibility = space.spaceRoom.visibility,
                     heroes = space.spaceRoom.heroes.toImmutableList(),
                     numberOfMembers = space.spaceRoom.numJoinedMembers,
-                    numberOfRooms = space.spaceRoom.childrenCount,
                 )
             }
         }

features/joinroom/impl/src/main/kotlin/io/element/android/features/joinroom/impl/JoinRoomView.kt

@@ -74,7 +74,7 @@ import io.element.android.libraries.designsystem.theme.components.TextField
 import io.element.android.libraries.designsystem.theme.components.TopAppBar
 import io.element.android.libraries.designsystem.theme.placeholderBackground
 import io.element.android.libraries.matrix.api.core.RoomIdOrAlias
-import io.element.android.libraries.matrix.api.room.join.JoinRule
+import io.element.android.libraries.matrix.api.spaces.SpaceRoomVisibility
 import io.element.android.libraries.matrix.ui.components.SpaceInfoRow
 import io.element.android.libraries.matrix.ui.components.SpaceMembersView
 import io.element.android.libraries.matrix.ui.model.InviteSender
@@ -567,10 +567,7 @@ private fun DefaultLoadedContent(
         subtitle = {
             when {
                 contentState.details is LoadedDetails.Space -> {
-                    SpaceInfoRow(
+                    SpaceInfoRow(visibility = SpaceRoomVisibility.fromJoinRule(contentState.joinRule))
-                        joinRule = contentState.joinRule ?: JoinRule.Public,
-                        numberOfRooms = contentState.details.childrenCount,
-                    )
                 }
                 contentState.alias != null -> {
                     RoomPreviewSubtitleAtom(contentState.alias.value)

features/space/impl/src/main/kotlin/io/element/android/features/space/impl/root/SpaceStateProvider.kt

@@ -13,8 +13,10 @@ import io.element.android.features.invite.api.acceptdecline.anAcceptDeclineInvit
 import io.element.android.libraries.architecture.AsyncAction
 import io.element.android.libraries.matrix.api.core.RoomId
 import io.element.android.libraries.matrix.api.room.CurrentUserMembership
+import io.element.android.libraries.matrix.api.room.join.JoinRule
 import io.element.android.libraries.matrix.api.spaces.SpaceRoom
 import io.element.android.libraries.previewutils.room.aSpaceRoom
+import kotlinx.collections.immutable.persistentListOf
 import kotlinx.collections.immutable.toImmutableList
 import kotlinx.collections.immutable.toImmutableMap
 import kotlinx.collections.immutable.toImmutableSet
@@ -23,6 +25,16 @@ open class SpaceStateProvider : PreviewParameterProvider<SpaceState> {
     override val values: Sequence<SpaceState>
         get() = sequenceOf(
             aSpaceState(),
+            aSpaceState(
+                parentSpace = aSpaceRoom(
+                    joinRule = JoinRule.Public
+                )
+            ),
+            aSpaceState(
+                parentSpace = aSpaceRoom(
+                    joinRule = JoinRule.Restricted(persistentListOf())
+                )
+            ),
             aSpaceState(
                 parentSpace = aSpaceRoom(
                     rawName = null,

features/space/impl/src/main/kotlin/io/element/android/features/space/impl/root/SpaceView.kt

@@ -134,10 +134,9 @@ private fun SpaceViewContent(
                     avatarData = currentSpace.getAvatarData(AvatarSize.SpaceHeader),
                     name = currentSpace.name,
                     topic = currentSpace.topic,
-                    joinRule = currentSpace.joinRule,
+                    visibility = currentSpace.visibility,
                     heroes = currentSpace.heroes.toImmutableList(),
                     numberOfMembers = currentSpace.numJoinedMembers,
-                    numberOfRooms = currentSpace.childrenCount,
                 )
             }
         }

libraries/matrix/api/src/main/kotlin/io/element/android/libraries/matrix/api/spaces/SpaceRoom.kt

@@ -46,4 +46,6 @@ data class SpaceRoom(
     } else {
         rawName
     }
+
+    val visibility = SpaceRoomVisibility.fromJoinRule(joinRule)
 }

libraries/matrix/api/src/main/kotlin/io/element/android/libraries/matrix/api/spaces/SpaceRoomVisibility.kt

@@ -0,0 +1,26 @@
+/*
+ * Copyright 2025 New Vector Ltd.
+ *
+ * SPDX-License-Identifier: AGPL-3.0-only OR LicenseRef-Element-Commercial
+ * Please see LICENSE files in the repository root for full details.
+ */
+
+package io.element.android.libraries.matrix.api.spaces
+
+import androidx.compose.runtime.Immutable
+import io.element.android.libraries.matrix.api.room.join.JoinRule
+@Immutable
+sealed interface SpaceRoomVisibility {
+    data object Private : SpaceRoomVisibility
+    data object Public : SpaceRoomVisibility
+    data object Restricted : SpaceRoomVisibility
+
+    companion object {
+        fun fromJoinRule(joinRule: JoinRule?): SpaceRoomVisibility = when (joinRule) {
+            JoinRule.Public -> Public
+            is JoinRule.Restricted, is JoinRule.KnockRestricted -> Restricted
+            // Else fallback to Private
+            else -> Private
+        }
+    }
+}

libraries/matrixui/src/main/kotlin/io/element/android/libraries/matrix/ui/components/SpaceHeaderRootView.kt

@@ -31,7 +31,6 @@ import io.element.android.libraries.ui.strings.CommonStrings
 @Composable
 fun SpaceHeaderRootView(
     numberOfSpaces: Int,
-    numberOfRooms: Int,
     modifier: Modifier = Modifier,
 ) {
     Column(
@@ -52,7 +51,7 @@ fun SpaceHeaderRootView(
         )
         SpaceInfoRow(
             leftText = numberOfSpaces(numberOfSpaces),
-            rightText = numberOfRooms(numberOfRooms),
+            rightText = null,
         )
         Text(
             text = stringResource(CommonStrings.screen_space_list_description),
@@ -68,6 +67,5 @@ fun SpaceHeaderRootView(
 internal fun SpaceHeaderRootViewPreview() = ElementPreview {
     SpaceHeaderRootView(
         numberOfSpaces = 3,
-        numberOfRooms = 10,
     )
 }

libraries/matrixui/src/main/kotlin/io/element/android/libraries/matrix/ui/components/SpaceHeaderView.kt

@@ -24,7 +24,7 @@ import io.element.android.libraries.designsystem.components.avatar.AvatarType
 import io.element.android.libraries.designsystem.components.avatar.anAvatarData
 import io.element.android.libraries.designsystem.preview.ElementPreview
 import io.element.android.libraries.designsystem.preview.PreviewsDayNight
-import io.element.android.libraries.matrix.api.room.join.JoinRule
+import io.element.android.libraries.matrix.api.spaces.SpaceRoomVisibility
 import io.element.android.libraries.matrix.api.user.MatrixUser
 import io.element.android.libraries.ui.strings.CommonStrings
 import kotlinx.collections.immutable.ImmutableList
@@ -38,10 +38,9 @@ fun SpaceHeaderView(
     avatarData: AvatarData,
     name: String?,
     topic: String?,
-    joinRule: JoinRule?,
+    visibility: SpaceRoomVisibility,
     heroes: ImmutableList<MatrixUser>,
     numberOfMembers: Int,
-    numberOfRooms: Int,
     modifier: Modifier = Modifier,
     topicMaxLines: Int = Int.MAX_VALUE,
 ) {
@@ -64,12 +63,7 @@ fun SpaceHeaderView(
             }
         },
         subtitle = {
-            if (joinRule != null) {
+            SpaceInfoRow(visibility = visibility)
-                SpaceInfoRow(
-                    joinRule = joinRule,
-                    numberOfRooms = numberOfRooms,
-                )
-            }
         },
         description = if (topic.isNullOrBlank()) {
             null
@@ -97,7 +91,7 @@ internal fun SpaceHeaderViewPreview() = ElementPreview {
         name = "Space name",
         topic = "Space topic: " + LoremIpsum(40).values.first(),
         topicMaxLines = 2,
-        joinRule = JoinRule.Public,
+        visibility = SpaceRoomVisibility.Public,
         heroes = persistentListOf(
             aMatrixUser(id = "@1:d", displayName = "Alice", avatarUrl = "aUrl"),
             aMatrixUser(id = "@2:d", displayName = "Bob"),
@@ -105,6 +99,5 @@ internal fun SpaceHeaderViewPreview() = ElementPreview {
             aMatrixUser(id = "@4:d", displayName = "Dave"),
         ),
         numberOfMembers = 999,
-        numberOfRooms = 10,
     )
 }

libraries/matrixui/src/main/kotlin/io/element/android/libraries/matrix/ui/components/SpaceInfoRow.kt

@@ -27,14 +27,16 @@ import io.element.android.compound.tokens.generated.CompoundIcons
 import io.element.android.libraries.designsystem.preview.ElementPreview
 import io.element.android.libraries.designsystem.preview.PreviewsDayNight
 import io.element.android.libraries.designsystem.theme.components.Icon
-import io.element.android.libraries.matrix.api.room.join.JoinRule
+import io.element.android.libraries.matrix.api.spaces.SpaceRoomVisibility
+import io.element.android.libraries.matrix.ui.model.icon
+import io.element.android.libraries.matrix.ui.model.label
 import io.element.android.libraries.ui.strings.CommonPlurals
 import io.element.android.libraries.ui.strings.CommonStrings
 
 @Composable
 fun SpaceInfoRow(
     leftText: String,
-    rightText: String,
+    rightText: String?,
     modifier: Modifier = Modifier,
     iconVector: ImageVector? = null,
 ) {
@@ -51,7 +53,11 @@ fun SpaceInfoRow(
                 tint = ElementTheme.colors.iconTertiary,
             )
         }
-        val text = stringResource(id = CommonStrings.screen_space_list_details, leftText, rightText)
+        val text = if (rightText != null) {
+            stringResource(id = CommonStrings.screen_space_list_details, leftText, rightText)
+        } else {
+            leftText
+        }
         Text(
             text = text,
             style = ElementTheme.typography.fontBodyLgRegular,
@@ -63,34 +69,14 @@ fun SpaceInfoRow(
 
 @Composable
 fun SpaceInfoRow(
-    joinRule: JoinRule,
+    visibility: SpaceRoomVisibility,
-    numberOfRooms: Int,
     modifier: Modifier = Modifier,
 ) {
-    val (leftText, rightText, icon) = when (joinRule) {
-        JoinRule.Public -> Triple(
-            stringResource(id = CommonStrings.common_public_space),
-            numberOfRooms(numberOfRooms),
-            CompoundIcons.Public(),
-        )
-        // TODO External space
-        // JoinRule.Private -> Triple(
-        //     stringResource(id = CommonStrings.common_external_space),
-        //     numberOfRooms(numberOfRooms),
-        //     CompoundIcons.Guest(),
-        // )
-        // JoinRule.Private,
-        else -> Triple(
-            stringResource(id = CommonStrings.common_private_space),
-            numberOfRooms(numberOfRooms),
-            CompoundIcons.Lock(),
-        )
-    }
     SpaceInfoRow(
-        leftText = leftText,
+        leftText = visibility.label,
-        rightText = rightText,
+        rightText = null,
         modifier = modifier,
-        iconVector = icon,
+        iconVector = visibility.icon,
     )
 }
 
@@ -124,12 +110,13 @@ internal fun SpaceInfoRowPreview() = ElementPreview {
             iconVector = CompoundIcons.Workspace(),
         )
         SpaceInfoRow(
-            joinRule = JoinRule.Private,
+            visibility = SpaceRoomVisibility.Private,
-            numberOfRooms = 4,
         )
         SpaceInfoRow(
-            joinRule = JoinRule.Public,
+            visibility = SpaceRoomVisibility.Public
-            numberOfRooms = 10,
+        )
+        SpaceInfoRow(
+            visibility = SpaceRoomVisibility.Restricted
         )
     }
 }

libraries/matrixui/src/main/kotlin/io/element/android/libraries/matrix/ui/components/SpaceRoomItemView.kt

@@ -34,7 +34,6 @@ import androidx.compose.ui.text.style.TextOverflow
 import androidx.compose.ui.tooling.preview.PreviewParameter
 import androidx.compose.ui.unit.dp
 import io.element.android.compound.theme.ElementTheme
-import io.element.android.compound.tokens.generated.CompoundIcons
 import io.element.android.libraries.designsystem.atomic.atoms.UnreadIndicatorAtom
 import io.element.android.libraries.designsystem.atomic.molecules.InviteButtonsRowMolecule
 import io.element.android.libraries.designsystem.components.avatar.Avatar
@@ -48,9 +47,11 @@ import io.element.android.libraries.designsystem.theme.components.Icon
 import io.element.android.libraries.designsystem.theme.components.Text
 import io.element.android.libraries.designsystem.theme.unreadIndicator
 import io.element.android.libraries.matrix.api.room.CurrentUserMembership
-import io.element.android.libraries.matrix.api.room.join.JoinRule
 import io.element.android.libraries.matrix.api.spaces.SpaceRoom
+import io.element.android.libraries.matrix.api.spaces.SpaceRoomVisibility
 import io.element.android.libraries.matrix.ui.model.getAvatarData
+import io.element.android.libraries.matrix.ui.model.icon
+import io.element.android.libraries.matrix.ui.model.label
 import io.element.android.libraries.ui.strings.CommonPlurals
 import io.element.android.libraries.ui.strings.CommonStrings
 
@@ -117,8 +118,8 @@ private fun SubtitleRow(
         if (visibilityIcon != null) {
             Icon(
                 modifier = Modifier
-                    .size(16.dp)
+                        .size(16.dp)
-                    .padding(end = 4.dp),
+                        .padding(end = 4.dp),
                 imageVector = visibilityIcon,
                 contentDescription = null,
                 tint = ElementTheme.colors.iconTertiary,
@@ -176,20 +177,20 @@ private fun SpaceRoomItemScaffold(
     content: @Composable ColumnScope.() -> Unit,
 ) {
     val clickModifier = Modifier
-        .combinedClickable(
+            .combinedClickable(
-            onClick = onClick,
+                    onClick = onClick,
-            onLongClick = onLongClick,
+                    onLongClick = onLongClick,
-            onLongClickLabel = stringResource(CommonStrings.action_open_context_menu),
+                    onLongClickLabel = stringResource(CommonStrings.action_open_context_menu),
-            indication = ripple(),
+                    indication = ripple(),
-            interactionSource = remember { MutableInteractionSource() }
+                    interactionSource = remember { MutableInteractionSource() }
-        )
+            )
-        .onKeyboardContextMenuAction { onLongClick }
+            .onKeyboardContextMenuAction { onLongClick }
     Row(
         modifier = modifier
-            .fillMaxWidth()
+                .fillMaxWidth()
-            .then(clickModifier)
+                .then(clickModifier)
-            .padding(horizontal = 16.dp, vertical = 8.dp)
+                .padding(horizontal = 16.dp, vertical = 8.dp)
-            .height(IntrinsicSize.Min),
+                .height(IntrinsicSize.Min),
     ) {
         Avatar(
             avatarData = avatarData,
@@ -212,11 +213,7 @@ private fun SpaceRoomItemScaffold(
 @ReadOnlyComposable
 private fun SpaceRoom.subtitle(): String {
     return if (isSpace) {
-        if (joinRule == JoinRule.Public) {
+        visibility.label
-            stringResource(CommonStrings.common_public_space)
-        } else {
-            stringResource(CommonStrings.common_private_space)
-        }
     } else {
         pluralStringResource(CommonPlurals.common_member_count, numJoinedMembers, numJoinedMembers)
     }
@@ -226,11 +223,7 @@ private fun SpaceRoom.subtitle(): String {
 @ReadOnlyComposable
 private fun SpaceRoom.info(): String {
     return if (isSpace) {
-        stringResource(
+        pluralStringResource(CommonPlurals.common_member_count, numJoinedMembers, numJoinedMembers)
-            CommonStrings.screen_space_list_details,
-            pluralStringResource(CommonPlurals.common_rooms, childrenCount, childrenCount),
-            pluralStringResource(CommonPlurals.common_member_count, numJoinedMembers, numJoinedMembers),
-        )
     } else {
         topic.orEmpty()
     }
@@ -238,10 +231,11 @@ private fun SpaceRoom.info(): String {
 
 @Composable
 private fun SpaceRoom.visibilityIcon(): ImageVector? {
-    return if (joinRule == JoinRule.Public) {
+    // Don't show any icon for restricted rooms as it's the default and would add noise
-        CompoundIcons.Public()
+    return if (visibility == SpaceRoomVisibility.Restricted) {
+        null
     } else {
-        CompoundIcons.LockSolid()
+        visibility.icon
     }
 }
 

libraries/matrixui/src/main/kotlin/io/element/android/libraries/matrix/ui/model/SpaceExtension.kt

@@ -7,9 +7,16 @@
 
 package io.element.android.libraries.matrix.ui.model
 
+import androidx.compose.runtime.Composable
+import androidx.compose.runtime.ReadOnlyComposable
+import androidx.compose.ui.graphics.vector.ImageVector
+import androidx.compose.ui.res.stringResource
+import io.element.android.compound.tokens.generated.CompoundIcons
 import io.element.android.libraries.designsystem.components.avatar.AvatarData
 import io.element.android.libraries.designsystem.components.avatar.AvatarSize
 import io.element.android.libraries.matrix.api.spaces.SpaceRoom
+import io.element.android.libraries.matrix.api.spaces.SpaceRoomVisibility
+import io.element.android.libraries.ui.strings.CommonStrings
 
 fun SpaceRoom.getAvatarData(size: AvatarSize) = AvatarData(
     id = roomId.value,
@@ -17,3 +24,24 @@ fun SpaceRoom.getAvatarData(size: AvatarSize) = AvatarData(
     url = avatarUrl,
     size = size,
 )
+
+val SpaceRoomVisibility.icon: ImageVector
+    @Composable
+    get() {
+        return when (this) {
+            SpaceRoomVisibility.Private -> CompoundIcons.LockSolid()
+            SpaceRoomVisibility.Public -> CompoundIcons.Public()
+            SpaceRoomVisibility.Restricted -> CompoundIcons.Workspace()
+        }
+    }
+
+val SpaceRoomVisibility.label: String
+    @Composable
+    @ReadOnlyComposable
+    get() {
+        return when (this) {
+            SpaceRoomVisibility.Private -> stringResource(CommonStrings.common_private_space)
+            SpaceRoomVisibility.Public -> stringResource(CommonStrings.common_public_space)
+            SpaceRoomVisibility.Restricted -> stringResource(CommonStrings.common_shared_space)
+        }
+    }

libraries/ui-strings/src/main/res/values/localazy.xml

@@ -94,6 +94,7 @@
   <string name="action_forgot_password">"Forgot password?"</string>
   <string name="action_forward">"Forward"</string>
   <string name="action_go_back">"Go back"</string>
+  <string name="action_go_to_settings">"Go to settings"</string>
   <string name="action_ignore">"Ignore"</string>
   <string name="action_invite">"Invite"</string>
   <string name="action_invite_friends">"Invite people"</string>
@@ -319,6 +320,7 @@ Reason: %1$s."</string>
   <string name="common_settings">"Settings"</string>
   <string name="common_share_space">"Share space"</string>
   <string name="common_shared_location">"Shared location"</string>
+  <string name="common_shared_space">"Shared space"</string>
   <string name="common_signing_out">"Signing out"</string>
   <string name="common_something_went_wrong">"Something went wrong"</string>
   <string name="common_something_went_wrong_message">"We encountered an issue. Please try again."</string>
@@ -427,6 +429,7 @@ Are you sure you want to continue?"</string>
   <string name="screen_create_poll_remove_accessibility_label">"Remove %1$s"</string>
   <string name="screen_create_poll_settings_section_title">"Settings"</string>
   <string name="screen_labs_enable_threads">"Enable thread replies"</string>
+  <string name="screen_labs_enable_threads_description">"Restarting the app is required to apply changes"</string>
   <string name="screen_labs_header_description">"Try out our latest ideas in development. These features are not finalised; they may be unstable, may change."</string>
   <string name="screen_labs_header_title">"Feeling experimental?"</string>
   <string name="screen_labs_title">"Labs"</string>