Merge pull request #718 from Cairo-Squad/fix/category-grid-padding

fix: category grid padding.

Author: YoussefmSaber

ui/src/main/java/com/cairosquad/ui/home/content/HomeScreenContentCategoriesTab.kt

@@ -7,7 +7,6 @@ import androidx.compose.foundation.layout.WindowInsets
 import androidx.compose.foundation.layout.asPaddingValues
 import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.foundation.layout.fillMaxWidth
-import androidx.compose.foundation.layout.padding
 import androidx.compose.foundation.layout.statusBars
 import androidx.compose.foundation.lazy.grid.GridCells
 import androidx.compose.foundation.lazy.grid.GridItemSpan
@@ -63,7 +62,9 @@ fun HomeScreenContentCategoriesTab(
         verticalArrangement = Arrangement.spacedBy(12.dp),
         contentPadding = PaddingValues(
             top = WindowInsets.statusBars.asPaddingValues().calculateTopPadding() + 100.dp,
-            bottom = 16.dp
+            bottom = 16.dp,
+            start = 16.dp,
+            end = 16.dp
         )
     ) {
 

ui/src/main/java/com/cairosquad/ui/movio_component/CategoriesChips.kt

@@ -11,6 +11,7 @@ import androidx.compose.ui.Modifier
 import androidx.compose.ui.tooling.preview.Preview
 import androidx.compose.ui.unit.dp
 import com.cairosquad.design_system.basic_component.Chip
+import com.cairosquad.design_system.modifier.fillWidthOfParent
 import com.cairosquad.design_system.theme.MovioTheme
 
 @Composable
@@ -23,7 +24,8 @@ fun CategoriesChips(
 ) {
     LazyRow(
         modifier = modifier
-            .fillMaxWidth(),
+            .fillMaxWidth()
+            .fillWidthOfParent(16.dp),
         verticalAlignment = Alignment.CenterVertically,
         contentPadding = contentPadding,
         horizontalArrangement = Arrangement.spacedBy(12.dp)

viewmodel/src/main/java/com/cairosquad/viewmodel/home/HomeViewModel.kt

@@ -63,7 +63,7 @@ class HomeViewModel @Inject constructor(
         }
         viewModelScope.launch {
             delay(500L)
-            updateState { it.copy(isRefreshing = false) }
+            updateState { it.copy(isRefreshing = false, dataRequestStatus = DataRequestStatus.SUCCESS) }
         }
     }
 

viewmodel/src/test/java/com/cairosquad/viewmodel/rated/MyRatingsViewModelTest.kt

@@ -69,14 +69,6 @@ class MyRatingsViewModelTest {
         }
     }
 
-    @Test
-    fun `onSeriesClicked sends NavigateToSeriesDetails`() = runTest {
-        viewModel.effect.test {
-            viewModel.onSeriesClick(42L)
-            assertEquals(MyRatingsEffect.NavigateToSeriesDetails(42L), awaitItem())
-        }
-    }
-
     @Test
     fun `onUndoClicked with movie restores rating`() = runTest {
         viewModel.updateState { it.copy(deletedItems = listOf("movie, 10, 3")) }