From a7e341041f5b76e9837142ad74eb9b90cc6ec751 Mon Sep 17 00:00:00 2001
From: Gibran Chevalley <gibran.chevalley@infomaniak.com>
Date: Wed, 4 Dec 2024 12:58:46 +0100
Subject: [PATCH] style: Clean OnboardingScreen

---
 .../ui/screen/onboarding/OnboardingScreen.kt  | 19 ++++++-------------
 1 file changed, 6 insertions(+), 13 deletions(-)

diff --git a/app/src/main/java/com/infomaniak/swisstransfer/ui/screen/onboarding/OnboardingScreen.kt b/app/src/main/java/com/infomaniak/swisstransfer/ui/screen/onboarding/OnboardingScreen.kt
index f7fd48dbb..7df7efb39 100644
--- a/app/src/main/java/com/infomaniak/swisstransfer/ui/screen/onboarding/OnboardingScreen.kt
+++ b/app/src/main/java/com/infomaniak/swisstransfer/ui/screen/onboarding/OnboardingScreen.kt
@@ -67,19 +67,17 @@ fun OnboardingScreen(goToMainActivity: () -> Unit) {
     }
 
     val onboardingPages = buildList {
-        Page.entries.forEach { page ->
-            add(page.toOnboardingPage(isHighlighted))
-        }
+        Page.entries.forEach { page -> add(page.toOnboardingPage(isHighlighted)) }
     }
 
     OnboardingScaffold(
         pagerState = pagerState,
         onboardingPages = onboardingPages,
-        bottomContent = {
+        bottomContent = { paddingValues ->
             BottomContent(
                 modifier = Modifier
-                    .padding(it)
-                    .consumeWindowInsets(it),
+                    .padding(paddingValues)
+                    .consumeWindowInsets(paddingValues),
                 isLastPage = { isLastPage },
                 startMainActivity = goToMainActivity,
                 goToNextPage = { coroutineScope.launch { pagerState.animateScrollToPage(pagerState.currentPage + 1) } },
@@ -91,7 +89,7 @@ fun OnboardingScreen(goToMainActivity: () -> Unit) {
 @Composable
 private fun Page.toOnboardingPage(isHighlighted: Map<Page, MutableState<Boolean>>) = OnboardingPage(
     background = background.image(),
-    illustration = { Illustration(illustration) },
+    illustration = { Image(illustration.image(), contentDescription = null) },
     text = {
         TitleAndDescription(
             page = this,
@@ -100,11 +98,6 @@ private fun Page.toOnboardingPage(isHighlighted: Map<Page, MutableState<Boolean>
     }
 )
 
-@Composable
-private fun Illustration(illustration: ThemedImage) {
-    Image(illustration.image(), contentDescription = null)
-}
-
 @Composable
 private fun TitleAndDescription(page: Page, isHighlighted: () -> Boolean) {
     Column(horizontalAlignment = Alignment.CenterHorizontally) {
@@ -132,7 +125,7 @@ private fun BottomContent(
     modifier: Modifier = Modifier,
     isLastPage: () -> Boolean,
     startMainActivity: () -> Unit,
-    goToNextPage: () -> Unit
+    goToNextPage: () -> Unit,
 ) {
     Box(
         contentAlignment = Alignment.Center,