Merge pull request #19 from Infomaniak/fix-adaptative-navigation

Fix adaptative navigation
Infomaniak · Aug 15, 2024 · d16272d · d16272d
2 parents 322851f + b73a041
commit d16272d
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/app/src/main/java/com/infomaniak/swisstransfer/ui/navigation/NavigationDestination.kt b/app/src/main/java/com/infomaniak/swisstransfer/ui/navigation/NavigationDestination.kt
@@ -22,6 +22,7 @@ import android.os.Bundle
 import androidx.navigation.NavBackStackEntry
 import kotlinx.serialization.Serializable
 import kotlin.reflect.KClass
+import kotlin.reflect.full.primaryConstructor
 
 /**
  * Sealed class representing the navigation arguments for the main navigation flow.
@@ -96,8 +97,7 @@ sealed class NavigationDestination {
         }
 
         private fun <T : NavigationDestination> createInstance(kClass: KClass<T>, bundle: Bundle?): T? {
-            val primaryConstructor = kClass.constructors.firstOrNull()
-            return primaryConstructor?.let {
+            return kClass.primaryConstructor?.let {
                 val args = it.parameters.associateWith { parameter ->
                     bundle?.get(parameter.name)
                 }

diff --git a/...java/com/infomaniak/swisstransfer/ui/screen/main/components/AppNavigationSuiteScaffold.kt b/...java/com/infomaniak/swisstransfer/ui/screen/main/components/AppNavigationSuiteScaffold.kt
@@ -65,10 +65,14 @@ fun AppNavigationSuiteScaffold(
     Surface(color = SwissTransferTheme.materialColors.background) {
         NavigationSuiteScaffoldLayout(
             navigationSuite = {
-                if (layoutType == NavigationSuiteType.NavigationBar) {
-                    AppNavigationBar(navigationItems, currentDestination, navigateToSelectedItem)
-                } else {
-                    AppNavigationRail(navigationItems, currentDestination, navigateToSelectedItem)
+                when (layoutType) {
+                    NavigationSuiteType.None -> Unit
+                    NavigationSuiteType.NavigationBar -> {
+                        AppNavigationBar(navigationItems, currentDestination, navigateToSelectedItem)
+                    }
+                    else -> {
+                        AppNavigationRail(navigationItems, currentDestination, navigateToSelectedItem)
+                    }
                 }
             },
             layoutType = layoutType,