From a752b047e98c44730b120c2a54ce30d3916379d5 Mon Sep 17 00:00:00 2001
From: Siddharth Agarwal <sagarwal@rtsl.org>
Date: Fri, 9 Aug 2024 15:41:20 +0530
Subject: [PATCH] Fix merge errors

---
 .../org/hisp/dhis/common/screens/Groups.kt    |  2 +-
 .../screens/others/NavigationBarScreen.kt     | 79 +++++++++----------
 2 files changed, 37 insertions(+), 44 deletions(-)

diff --git a/common/src/commonMain/kotlin/org/hisp/dhis/common/screens/Groups.kt b/common/src/commonMain/kotlin/org/hisp/dhis/common/screens/Groups.kt
index 3d4eaefd1..403da1712 100644
--- a/common/src/commonMain/kotlin/org/hisp/dhis/common/screens/Groups.kt
+++ b/common/src/commonMain/kotlin/org/hisp/dhis/common/screens/Groups.kt
@@ -17,6 +17,6 @@ enum class Groups(val label: String) {
     METADATA_AVATAR("Metadata Avatar"),
     INDICATOR("Indicators"),
     PARAMETER_SELECTOR("Parameter selector"),
-    NO_GROUP_SELECTED("No group selected"),
     NAVIGATION_BAR("Navigation Bar"),
+    NO_GROUP_SELECTED("No group selected"),
 }
diff --git a/common/src/commonMain/kotlin/org/hisp/dhis/common/screens/others/NavigationBarScreen.kt b/common/src/commonMain/kotlin/org/hisp/dhis/common/screens/others/NavigationBarScreen.kt
index 657180dd5..2feaf8f9f 100644
--- a/common/src/commonMain/kotlin/org/hisp/dhis/common/screens/others/NavigationBarScreen.kt
+++ b/common/src/commonMain/kotlin/org/hisp/dhis/common/screens/others/NavigationBarScreen.kt
@@ -1,8 +1,5 @@
 package org.hisp.dhis.common.screens.others
 
-import androidx.compose.foundation.background
-import androidx.compose.foundation.layout.Spacer
-import androidx.compose.foundation.layout.height
 import androidx.compose.material.icons.Icons
 import androidx.compose.material.icons.automirrored.filled.Assignment
 import androidx.compose.material.icons.automirrored.filled.List
@@ -24,13 +21,11 @@ import androidx.compose.runtime.getValue
 import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.remember
 import androidx.compose.runtime.setValue
-import androidx.compose.ui.Modifier
+import org.hisp.dhis.common.screens.Groups
 import org.hisp.dhis.mobile.ui.designsystem.component.ColumnComponentContainer
-import org.hisp.dhis.mobile.ui.designsystem.component.SubTitle
+import org.hisp.dhis.mobile.ui.designsystem.component.ColumnScreenContainer
 import org.hisp.dhis.mobile.ui.designsystem.component.navigationBar.NavigationBar
 import org.hisp.dhis.mobile.ui.designsystem.component.navigationBar.NavigationBarItem
-import org.hisp.dhis.mobile.ui.designsystem.theme.Spacing
-import org.hisp.dhis.mobile.ui.designsystem.theme.SurfaceColor
 
 @Composable
 fun NavigationBarScreen() {
@@ -160,48 +155,46 @@ fun NavigationBarScreen() {
         ),
     )
 
-    ColumnComponentContainer(
-        title = "Navigation Bar",
-        modifier = Modifier
-            .background(SurfaceColor.Container),
+    ColumnScreenContainer(
+        title = Groups.NAVIGATION_BAR.label,
     ) {
-        SubTitle("Home")
-        var selectedHomeItemIndex by remember { mutableStateOf<Int?>(null) }
-        NavigationBar(
-            items = homeItems,
-            selectedItemIndex = selectedHomeItemIndex,
-        ) {
-            selectedHomeItemIndex = it
+        ColumnComponentContainer("Home") {
+            var selectedHomeItemIndex by remember { mutableStateOf<Int?>(null) }
+            NavigationBar(
+                items = homeItems,
+                selectedItemIndex = selectedHomeItemIndex,
+            ) {
+                selectedHomeItemIndex = it
+            }
         }
-
-        Spacer(modifier = Modifier.height(Spacing.Spacing24))
-        SubTitle("Program dashboard")
-        var selectedProgramItemIndex by remember { mutableStateOf<Int?>(null) }
-        NavigationBar(
-            items = programItems,
-            selectedItemIndex = selectedProgramItemIndex,
-        ) {
-            selectedProgramItemIndex = it
+        ColumnComponentContainer("Program dashboard") {
+            var selectedProgramItemIndex by remember { mutableStateOf<Int?>(null) }
+            NavigationBar(
+                items = programItems,
+                selectedItemIndex = selectedProgramItemIndex,
+            ) {
+                selectedProgramItemIndex = it
+            }
         }
 
-        Spacer(modifier = Modifier.height(Spacing.Spacing24))
-        SubTitle("Enrollment dashboard")
-        var selectedEnrollmentItemIndex by remember { mutableStateOf<Int?>(null) }
-        NavigationBar(
-            items = enrollmentItems,
-            selectedItemIndex = selectedEnrollmentItemIndex,
-        ) {
-            selectedEnrollmentItemIndex = it
+        ColumnComponentContainer("Enrollment dashboard") {
+            var selectedEnrollmentItemIndex by remember { mutableStateOf<Int?>(null) }
+            NavigationBar(
+                items = enrollmentItems,
+                selectedItemIndex = selectedEnrollmentItemIndex,
+            ) {
+                selectedEnrollmentItemIndex = it
+            }
         }
 
-        Spacer(modifier = Modifier.height(Spacing.Spacing24))
-        SubTitle("Form")
-        var selectedFormItemIndex by remember { mutableStateOf<Int?>(null) }
-        NavigationBar(
-            items = formItems,
-            selectedItemIndex = selectedFormItemIndex,
-        ) {
-            selectedFormItemIndex = it
+        ColumnComponentContainer("Form") {
+            var selectedFormItemIndex by remember { mutableStateOf<Int?>(null) }
+            NavigationBar(
+                items = formItems,
+                selectedItemIndex = selectedFormItemIndex,
+            ) {
+                selectedFormItemIndex = it
+            }
         }
     }
 }