Fix: improve offline indicator designs (#341)

* Use the right font weight for the offline indicator title.

* Remove unneeded suppression in some build files.

features/invitelist/impl/src/main/kotlin/io/element/android/features/invitelist/impl/InviteListStateProvider.kt

@@ -39,7 +39,7 @@ internal fun aInviteListState() = InviteListState(
 internal fun aInviteListInviteSummaryList(): ImmutableList<InviteListInviteSummary> {
     return persistentListOf(
         InviteListInviteSummary(
-            roomId = RoomId("!id1"),
+            roomId = RoomId("!id1:example.com"),
             roomName = "Room 1",
             roomAlias = "#room:example.org",
             sender = InviteSender(
@@ -48,7 +48,7 @@ internal fun aInviteListInviteSummaryList(): ImmutableList<InviteListInviteSumma
             ),
         ),
         InviteListInviteSummary(
-            roomId = RoomId("!id2"),
+            roomId = RoomId("!id2:example.com"),
             roomName = "Room 2",
             sender = InviteSender(
                 userId = UserId("@bob:example.org"),
@@ -56,7 +56,7 @@ internal fun aInviteListInviteSummaryList(): ImmutableList<InviteListInviteSumma
             ),
         ),
         InviteListInviteSummary(
-            roomId = RoomId("!id3"),
+            roomId = RoomId("!id3:example.com"),
             roomName = "Alice",
             roomAlias = "@alice:example.com"
         ),

features/invitelist/impl/src/main/kotlin/io/element/android/features/invitelist/impl/model/InviteListInviteSummaryProvider.kt

@@ -30,7 +30,7 @@ open class InviteListInviteSummaryProvider : PreviewParameterProvider<InviteList
 }
 
 fun aInviteListInviteSummary() = InviteListInviteSummary(
-    roomId = RoomId("!room1"),
+    roomId = RoomId("!room1:example.com"),
     roomName = "Some room",
     sender = InviteSender(
         userId = UserId("@alice:example.org"),

features/networkmonitor/api/build.gradle.kts

@@ -14,8 +14,6 @@
  * limitations under the License.
  */
 
-// TODO: Remove once https://youtrack.jetbrains.com/issue/KTIJ-19369 is fixed
-@Suppress("DSL_SCOPE_VIOLATION")
 plugins {
     id("io.element.android-compose-library")
 }

features/networkmonitor/api/src/main/kotlin/io/element/android/features/networkmonitor/api/ui/ConnectivityIndicatorView.kt

@@ -37,14 +37,12 @@ import androidx.compose.material.icons.Icons
 import androidx.compose.material.icons.outlined.WifiOff
 import androidx.compose.material3.MaterialTheme
 import androidx.compose.runtime.Composable
-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
 import androidx.compose.ui.graphics.ColorFilter
 import androidx.compose.ui.res.stringResource
+import androidx.compose.ui.text.font.FontWeight
 import androidx.compose.ui.tooling.preview.Preview
 import androidx.compose.ui.unit.dp
 import io.element.android.libraries.designsystem.ElementTextStyles
@@ -99,7 +97,11 @@ private fun Indicator(modifier: Modifier = Modifier) {
             modifier = Modifier.size(16.dp),
         )
         Spacer(modifier = Modifier.width(8.dp))
-        Text(text = stringResource(StringR.string.common_offline), style = ElementTextStyles.Regular.bodyMD, color = tint)
+        Text(
+            text = stringResource(StringR.string.common_offline),
+            style = ElementTextStyles.Regular.bodyMD.copy(fontWeight = FontWeight.Medium),
+            color = tint,
+        )
     }
 }
 

features/networkmonitor/impl/build.gradle.kts

@@ -14,8 +14,6 @@
  * limitations under the License.
  */
 
-// TODO: Remove once https://youtrack.jetbrains.com/issue/KTIJ-19369 is fixed
-@Suppress("DSL_SCOPE_VIOLATION")
 plugins {
     id("io.element.android-library")
     alias(libs.plugins.anvil)

features/networkmonitor/test/build.gradle.kts

@@ -14,8 +14,6 @@
  * limitations under the License.
  */
 
-// TODO: Remove once https://youtrack.jetbrains.com/issue/KTIJ-19369 is fixed
-@Suppress("DSL_SCOPE_VIOLATION")
 plugins {
     id("io.element.android-library")
 }

libraries/featureflag/impl/build.gradle.kts

@@ -14,8 +14,6 @@
  * limitations under the License.
  */
 
-// TODO: Remove once https://youtrack.jetbrains.com/issue/KTIJ-19369 is fixed
-@Suppress("DSL_SCOPE_VIOLATION")
 plugins {
     id("io.element.android-library")
     alias(libs.plugins.anvil)

libraries/featureflag/ui/build.gradle.kts

@@ -14,8 +14,6 @@
  * limitations under the License.
  */
 
-// TODO: Remove once https://youtrack.jetbrains.com/issue/KTIJ-19369 is fixed
-@Suppress("DSL_SCOPE_VIOLATION")
 plugins {
     id("io.element.android-compose-library")
     alias(libs.plugins.anvil)