From 5875de1c51f4151dd1f17816f8460456370ffe44 Mon Sep 17 00:00:00 2001
From: Benoit Marty <benoit@matrix.org>
Date: Thu, 9 Feb 2023 18:21:25 +0100
Subject: [PATCH] Rename classes (not a State)

---
 .../features/onboarding/OnBoardingScreen.kt        |  8 ++++----
 ...plashCarouselState.kt => SplashCarouselData.kt} |  2 +-
 ...tateFactory.kt => SplashCarouselDataFactory.kt} | 14 +++++++-------
 3 files changed, 12 insertions(+), 12 deletions(-)
 rename features/onboarding/src/main/kotlin/io/element/android/features/onboarding/{SplashCarouselState.kt => SplashCarouselData.kt} (96%)
 rename features/onboarding/src/main/kotlin/io/element/android/features/onboarding/{SplashCarouselStateFactory.kt => SplashCarouselDataFactory.kt} (90%)

diff --git a/features/onboarding/src/main/kotlin/io/element/android/features/onboarding/OnBoardingScreen.kt b/features/onboarding/src/main/kotlin/io/element/android/features/onboarding/OnBoardingScreen.kt
index dc8362563e..b2699c1816 100644
--- a/features/onboarding/src/main/kotlin/io/element/android/features/onboarding/OnBoardingScreen.kt
+++ b/features/onboarding/src/main/kotlin/io/element/android/features/onboarding/OnBoardingScreen.kt
@@ -59,8 +59,8 @@ fun OnBoardingScreen(
     onSignUp: () -> Unit = {},
     onSignIn: () -> Unit = {},
 ) {
-    val carrouselState = remember { SplashCarouselStateFactory().create() }
-    val nbOfPages = carrouselState.items.size
+    val carrouselData = remember { SplashCarouselDataFactory().create() }
+    val nbOfPages = carrouselData.items.size
     var key by remember { mutableStateOf(false) }
     Box(
         modifier = modifier
@@ -92,7 +92,7 @@ fun OnBoardingScreen(
                 state = pagerState,
             ) { page ->
                 // Our page content
-                OnBoardingPage(carrouselState.items[page])
+                OnBoardingPage(carrouselData.items[page])
             }
             HorizontalPagerIndicator(
                 pagerState = pagerState,
@@ -118,7 +118,7 @@ fun OnBoardingScreen(
 
 @Composable
 fun OnBoardingPage(
-    item: SplashCarouselState.Item,
+    item: SplashCarouselData.Item,
     modifier: Modifier = Modifier,
 ) {
     Box(
diff --git a/features/onboarding/src/main/kotlin/io/element/android/features/onboarding/SplashCarouselState.kt b/features/onboarding/src/main/kotlin/io/element/android/features/onboarding/SplashCarouselData.kt
similarity index 96%
rename from features/onboarding/src/main/kotlin/io/element/android/features/onboarding/SplashCarouselState.kt
rename to features/onboarding/src/main/kotlin/io/element/android/features/onboarding/SplashCarouselData.kt
index f6523da7a6..58d1b6534c 100644
--- a/features/onboarding/src/main/kotlin/io/element/android/features/onboarding/SplashCarouselState.kt
+++ b/features/onboarding/src/main/kotlin/io/element/android/features/onboarding/SplashCarouselData.kt
@@ -19,7 +19,7 @@ package io.element.android.features.onboarding
 import androidx.annotation.DrawableRes
 import androidx.annotation.StringRes
 
-data class SplashCarouselState(
+data class SplashCarouselData(
     val items: List<Item>
 ) {
     data class Item(
diff --git a/features/onboarding/src/main/kotlin/io/element/android/features/onboarding/SplashCarouselStateFactory.kt b/features/onboarding/src/main/kotlin/io/element/android/features/onboarding/SplashCarouselDataFactory.kt
similarity index 90%
rename from features/onboarding/src/main/kotlin/io/element/android/features/onboarding/SplashCarouselStateFactory.kt
rename to features/onboarding/src/main/kotlin/io/element/android/features/onboarding/SplashCarouselDataFactory.kt
index fc06ba49b6..e2848839ce 100644
--- a/features/onboarding/src/main/kotlin/io/element/android/features/onboarding/SplashCarouselStateFactory.kt
+++ b/features/onboarding/src/main/kotlin/io/element/android/features/onboarding/SplashCarouselDataFactory.kt
@@ -19,8 +19,8 @@ package io.element.android.features.onboarding
 import androidx.annotation.DrawableRes
 import io.element.android.libraries.ui.strings.R as StringR
 
-class SplashCarouselStateFactory {
-    fun create(): SplashCarouselState {
+class SplashCarouselDataFactory {
+    fun create(): SplashCarouselData {
         val lightTheme = true
 
         fun background(@DrawableRes lightDrawable: Int) =
@@ -29,9 +29,9 @@ class SplashCarouselStateFactory {
         fun hero(@DrawableRes lightDrawable: Int, @DrawableRes darkDrawable: Int) =
             if (lightTheme) lightDrawable else darkDrawable
 
-        return SplashCarouselState(
+        return SplashCarouselData(
             listOf(
-                SplashCarouselState.Item(
+                SplashCarouselData.Item(
                     StringR.string.ftue_auth_carousel_secure_title,
                     StringR.string.ftue_auth_carousel_secure_body,
                     hero(
@@ -40,19 +40,19 @@ class SplashCarouselStateFactory {
                     ),
                     background(R.drawable.bg_carousel_page_1)
                 ),
-                SplashCarouselState.Item(
+                SplashCarouselData.Item(
                     StringR.string.ftue_auth_carousel_control_title,
                     StringR.string.ftue_auth_carousel_control_body,
                     hero(R.drawable.ic_splash_control, R.drawable.ic_splash_control_dark),
                     background(R.drawable.bg_carousel_page_2)
                 ),
-                SplashCarouselState.Item(
+                SplashCarouselData.Item(
                     StringR.string.ftue_auth_carousel_encrypted_title,
                     StringR.string.ftue_auth_carousel_encrypted_body,
                     hero(R.drawable.ic_splash_secure, R.drawable.ic_splash_secure_dark),
                     background(R.drawable.bg_carousel_page_3)
                 ),
-                SplashCarouselState.Item(
+                SplashCarouselData.Item(
                     collaborationTitle(),
                     StringR.string.ftue_auth_carousel_workplace_body,
                     hero(