#31 - PR's suggestions

Author: rviannaoliveira

core-shared-ui/src/main/java/com/codandotv/streamplayerapp/core_shared_ui/resources/Colors.kt

@@ -5,7 +5,7 @@ import androidx.compose.material3.lightColorScheme
 import androidx.compose.ui.graphics.Color
 
 object Colors {
-    val DarkDeepGray = Color(android.graphics.Color.parseColor("#262626"))
+    val DarkDeepGray = Color(0xFF262626)
 
     val LightColors = lightColorScheme(
         primary = Color(0xFFE50914),

core-shared-ui/src/main/java/com/codandotv/streamplayerapp/core_shared_ui/widget/IconWithText.kt

@@ -17,10 +17,15 @@ import androidx.compose.ui.unit.sp
 
 @Composable
 fun IconWithText(
-    imageVector: ImageVector, imageColor: Color, text: String, textColor: Color
+    imageVector: ImageVector,
+    imageColor: Color,
+    text: String,
+    textColor: Color,
+    modifier: Modifier = Modifier
 ) {
     Column(
         horizontalAlignment = Alignment.CenterHorizontally,
+        modifier = modifier
     ) {
         Icon(
             imageVector = imageVector,

feature-list-streams/src/main/java/com/codandotv/streamplayerapp/feature_list_streams/detail/presentation/screens/DetailStreamsScreen.kt

@@ -1,7 +1,6 @@
 package com.codandotv.streamplayerapp.feature_list_streams.detail.presentation.screens
 
 import android.annotation.SuppressLint
-import androidx.compose.foundation.Image
 import androidx.compose.foundation.layout.*
 import androidx.compose.material.icons.Icons
 import androidx.compose.material.icons.filled.*
@@ -13,7 +12,6 @@ import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.graphics.Color
 import androidx.compose.ui.platform.LocalLifecycleOwner
-import androidx.compose.ui.res.painterResource
 import androidx.compose.ui.res.stringResource
 import androidx.compose.ui.text.font.FontWeight
 import androidx.compose.ui.unit.dp
@@ -38,7 +36,7 @@ fun DetailStreamScreen(
     }
 
     val lifecycleOwner = LocalLifecycleOwner.current
-    lifecycle(lifecycleOwner, viewModel, disposable)
+    Lifecycle(lifecycleOwner, viewModel, disposable)
 
     Box(modifier = Modifier.fillMaxSize()) {
         when (val uiState = uiStateRemember.value) {
@@ -127,7 +125,7 @@ private fun SetupDetailScreen(
 }
 
 @Composable
-private fun lifecycle(
+private fun Lifecycle(
     lifecycleOwner: LifecycleOwner, viewModel: DetailStreamViewModel, disposable: () -> Unit
 ) {
     DisposableEffect(lifecycleOwner) {

feature-list-streams/src/main/java/com/codandotv/streamplayerapp/feature_list_streams/detail/presentation/widget/DetailStreamActionOption.kt

@@ -16,10 +16,11 @@ import com.codandotv.streamplayerapp.core_shared_ui.widget.IconWithText
 import com.codandotv.streamplayerapp.feature_list_streams.R
 
 @Composable
-fun DetailStreamActionOption() {
+fun DetailStreamActionOption(
+    modifier: Modifier = Modifier.fillMaxWidth()
+) {
     Row(
-        modifier = Modifier
-            .fillMaxWidth(),
+        modifier = modifier,
         horizontalArrangement = Arrangement.SpaceEvenly
     ) {
         IconWithText(

feature-list-streams/src/main/java/com/codandotv/streamplayerapp/feature_list_streams/detail/presentation/widget/DetailStreamButtonAction.kt

@@ -15,18 +15,18 @@ import androidx.compose.ui.unit.sp
 @Composable
 fun DetailStreamButtonAction(
     buttonsColors: ButtonColors,
-    imageVector : ImageVector,
-    imageVectorColor : Color,
-    text : String,
-    textColor : Color,
+    imageVector: ImageVector,
+    imageVectorColor: Color,
+    text: String,
+    textColor: Color,
+    modifier: Modifier = Modifier.fillMaxWidth(),
     onClick: () -> Unit,
 ) {
     Button(
         onClick = { },
         shape = RoundedCornerShape(4.dp),
-        modifier = Modifier
-            .fillMaxWidth(),
-        colors = buttonsColors
+        modifier = modifier,
+        colors = buttonsColors,
     ) {
         Row(
             verticalAlignment = Alignment.CenterVertically

feature-list-streams/src/main/java/com/codandotv/streamplayerapp/feature_list_streams/detail/presentation/widget/DetailStreamImagePreview.kt

@@ -19,15 +19,18 @@ import com.codandotv.streamplayerapp.feature_list_streams.R
 import com.codandotv.streamplayerapp.feature_list_streams.detail.presentation.screens.DetailStreamsUIState.DetailStreamsLoadedUIState
 
 @Composable
-fun DetailStreamImagePreview(uiState: DetailStreamsLoadedUIState) {
+fun DetailStreamImagePreview(
+    uiState: DetailStreamsLoadedUIState,
+    modifier: Modifier = Modifier
+        .fillMaxWidth()
+        .aspectRatio(16f / 9f)
+) {
     Box(contentAlignment = Alignment.Center) {
         AsyncImage(
             model = uiState.detailStream.url,
             contentScale = ContentScale.FillBounds,
             contentDescription = uiState.detailStream.tagline,
-            modifier = Modifier
-                .fillMaxWidth()
-                .aspectRatio(16f / 9f)
+            modifier = modifier
         )
         Box(
             modifier = Modifier

feature-list-streams/src/main/java/com/codandotv/streamplayerapp/feature_list_streams/detail/presentation/widget/DetailStreamRowHeader.kt

@@ -1,10 +1,7 @@
 package com.codandotv.streamplayerapp.feature_list_streams.detail.presentation.widget
 
 import androidx.compose.foundation.Image
-import androidx.compose.foundation.layout.Row
-import androidx.compose.foundation.layout.fillMaxWidth
-import androidx.compose.foundation.layout.offset
-import androidx.compose.foundation.layout.size
+import androidx.compose.foundation.layout.*
 import androidx.compose.material3.MaterialTheme
 import androidx.compose.material3.Text
 import androidx.compose.runtime.Composable
@@ -20,10 +17,12 @@ import androidx.compose.ui.unit.sp
 import com.codandotv.streamplayerapp.feature_list_streams.R
 
 @Composable
-fun DetailStreamRowHeader() {
+fun DetailStreamRowHeader(
+    modifier: Modifier = Modifier.fillMaxWidth()
+) {
     Row(
         verticalAlignment = Alignment.CenterVertically,
-        modifier = Modifier.fillMaxWidth()
+        modifier = modifier
     ) {
         Image(
             painter = painterResource(id = R.drawable.netflix_detail),