code review.

Author: commandiron

app/src/main/java/com/commandiron/composeloading/MainActivity.kt

@@ -72,7 +72,7 @@ fun ShowScreen() {
 
                 12 -> { FoldingCube(Modifier.background(StarCommandBlue).fillMaxSize()) }
 
-                13 -> { LoadingBar(fakeMillis = 10000) }
+                13 -> { LoadingBar(fakeMillis = 10000, fillColor = Bluetiful) }
 
                 14 -> { LoadingDots("Loading") }
             }

app/src/main/java/com/commandiron/composeloading/ui/theme/Theme.kt

@@ -8,13 +8,13 @@ import androidx.compose.runtime.Composable
 import androidx.compose.ui.graphics.Color
 
 private val DarkColorScheme = darkColorScheme(
-    primary = Bluetiful,
-    surface = Color.White,
+    primary = Color.White,
+    onPrimary = Color.Black
 )
 
 private val LightColorScheme = lightColorScheme(
-    primary = Bluetiful,
-    surface = Color.White
+    primary = Color.White,
+    onPrimary = Color.Black
 )
 
 @Composable

compose-loading/src/main/java/com/commandiron/compose_loading/ChasingDots.kt

@@ -24,7 +24,7 @@ fun ChasingDots(
     durationMillis: Int = 2000,
     delayBetweenDotsMillis: Int = 50,
     size: Dp = 30.dp,
-    color: Color = MaterialTheme.colorScheme.surface,
+    color: Color = MaterialTheme.colorScheme.primary,
     circleRatio: Float = 0.25f
 ) {
     val transition = rememberInfiniteTransition()

compose-loading/src/main/java/com/commandiron/compose_loading/ChasingTwoDots.kt

@@ -24,7 +24,7 @@ fun ChasingTwoDots(
     durationMillis: Int = 2000,
     durationBetweenDotsMillis: Int = 400,
     size: DpSize = DpSize(30.dp, 30.dp),
-    color: Color = MaterialTheme.colorScheme.surface,
+    color: Color = MaterialTheme.colorScheme.primary,
 ) {
     val transition = rememberInfiniteTransition()
 

compose-loading/src/main/java/com/commandiron/compose_loading/Circle.kt

@@ -19,7 +19,7 @@ fun Circle(
     size: Dp = 30.dp,
     durationMillis: Int = 1400,
     delayMillis: Int = 0,
-    color: Color = MaterialTheme.colorScheme.surface,
+    color: Color = MaterialTheme.colorScheme.primary,
     shape: Shape = CircleShape,
 ) {
     Box(

compose-loading/src/main/java/com/commandiron/compose_loading/DoubleBounce.kt

@@ -22,7 +22,7 @@ fun DoubleBounce(
     durationMillis: Int = 2000,
     delayMillis: Int = 0,
     size: DpSize = DpSize(30.dp, 30.dp),
-    color: Color = MaterialTheme.colorScheme.surface,
+    color: Color = MaterialTheme.colorScheme.primary,
     shape: Shape = CircleShape
 ) {
     val transition = rememberInfiniteTransition()

compose-loading/src/main/java/com/commandiron/compose_loading/LoadingBar.kt

@@ -34,7 +34,7 @@ fun LoadingBar(
     width: Dp = 200.dp,
     backgroundColor: Color = MaterialTheme.colorScheme.background,
     fillColor: Color = MaterialTheme.colorScheme.primary,
-    borderColor: Color = Color.Black
+    borderColor: Color = MaterialTheme.colorScheme.onPrimary
 ) {
     val loadingBarProgress = remember {
         Animatable(0f)

compose-loading/src/main/java/com/commandiron/compose_loading/Pulse.kt

@@ -21,7 +21,7 @@ fun Pulse(
     durationMillis: Int = 1000,
     delayMillis: Int = 0,
     size: DpSize = DpSize(30.dp, 30.dp),
-    color: Color = MaterialTheme.colorScheme.surface,
+    color: Color = MaterialTheme.colorScheme.primary,
     shape: Shape = CircleShape,
 ) {
     val transition = rememberInfiniteTransition()

compose-loading/src/main/java/com/commandiron/compose_loading/RotatingPlane.kt

@@ -25,7 +25,7 @@ fun RotatingPlane(
     durationMillis: Int = 1200,
     delayMillis: Int = 0,
     size: DpSize = DpSize(30.dp, 30.dp),
-    color: Color = MaterialTheme.colorScheme.surface,
+    color: Color = MaterialTheme.colorScheme.primary,
     shape: Shape = RectangleShape,
     contentOnPlane: @Composable BoxScope. () -> Unit = {}
 ) {

compose-loading/src/main/java/com/commandiron/compose_loading/ThreeBounce.kt

@@ -18,11 +18,11 @@ import com.commandiron.compose_loading.transition.fractionTransition
 @Composable
 fun ThreeBounce(
     modifier: Modifier = Modifier,
-    size: DpSize = DpSize(30.dp, 30.dp),
-    color: Color = MaterialTheme.colorScheme.surface,
-    shape: Shape = CircleShape,
     durationMillis: Int = 1400,
-    delayBetweenDotsMillis: Int = 160
+    delayBetweenDotsMillis: Int = 160,
+    size: DpSize = DpSize(30.dp, 30.dp),
+    color: Color = MaterialTheme.colorScheme.primary,
+    shape: Shape = CircleShape
 ) {
     val transition = rememberInfiniteTransition()