fix: changes according designer review

Author: PavloNetrebchuk

course/src/main/java/org/openedx/course/presentation/home/VideosHomePagerCardContent.kt

@@ -1,6 +1,5 @@
 package org.openedx.course.presentation.home
 
-import androidx.compose.foundation.BorderStroke
 import androidx.compose.foundation.layout.Column
 import androidx.compose.foundation.layout.Row
 import androidx.compose.foundation.layout.Spacer
@@ -10,7 +9,6 @@ import androidx.compose.foundation.layout.padding
 import androidx.compose.foundation.layout.size
 import androidx.compose.foundation.layout.width
 import androidx.compose.foundation.shape.CircleShape
-import androidx.compose.material.Card
 import androidx.compose.material.Icon
 import androidx.compose.material.LinearProgressIndicator
 import androidx.compose.material.MaterialTheme
@@ -26,9 +24,7 @@ import androidx.compose.ui.semantics.semantics
 import androidx.compose.ui.text.font.FontWeight
 import androidx.compose.ui.unit.dp
 import org.openedx.core.domain.model.Block
-import org.openedx.core.extension.getUnitChapter
 import org.openedx.core.ui.theme.appColors
-import org.openedx.core.ui.theme.appShapes
 import org.openedx.core.ui.theme.appTypography
 import org.openedx.course.R
 import org.openedx.course.presentation.contenttab.CourseContentVideoEmptyState
@@ -131,42 +127,21 @@ fun VideosHomePagerCardContent(
             Spacer(modifier = Modifier.height(8.dp))
 
             // Video card using CourseVideoItem
-            Card(
+            CourseVideoItem(
                 modifier = Modifier
-                    .fillMaxWidth(),
-                backgroundColor = MaterialTheme.appColors.cardViewBackground,
-                shape = MaterialTheme.appShapes.videoPreviewShape,
-                elevation = 0.dp,
-                border = BorderStroke(
-                    width = 1.dp,
-                    color = MaterialTheme.appColors.cardViewBorder
-                )
-            ) {
-                Column {
-                    CourseVideoItem(
-                        modifier = Modifier
-                            .fillMaxWidth()
-                            .height(180.dp),
-                        videoBlock = firstIncompleteVideo,
-                        preview = uiState.videoPreview,
-                        progress = videoProgress,
-                        onClick = {
-                            onVideoClick(firstIncompleteVideo)
-                        },
-                        titleStyle = MaterialTheme.appTypography.titleMedium,
-                        contentModifier = Modifier
-                            .padding(horizontal = 16.dp, vertical = 8.dp),
-                        progressModifier = Modifier.height(8.dp),
-                    )
-                    Text(
-                        modifier = Modifier.padding(horizontal = 16.dp, vertical = 8.dp),
-                        text = uiState.courseStructure.blockData
-                            .getUnitChapter(firstIncompleteVideo.id)?.displayName ?: "",
-                        style = MaterialTheme.appTypography.labelMedium,
-                        color = MaterialTheme.appColors.textPrimary,
-                    )
-                }
-            }
+                    .fillMaxWidth()
+                    .height(180.dp),
+                videoBlock = firstIncompleteVideo,
+                preview = uiState.videoPreview,
+                progress = videoProgress,
+                onClick = {
+                    onVideoClick(firstIncompleteVideo)
+                },
+                titleStyle = MaterialTheme.appTypography.titleMedium,
+                contentModifier = Modifier
+                    .padding(horizontal = 16.dp, vertical = 8.dp),
+                progressModifier = Modifier.height(8.dp),
+            )
         } else {
             CaughtUpMessage(
                 message = stringResource(R.string.course_videos_caught_up)

course/src/main/java/org/openedx/course/presentation/ui/CourseUI.kt

@@ -721,7 +721,8 @@ fun CourseVideoItem(
     contentModifier: Modifier = Modifier.padding(8.dp),
     progressModifier: Modifier = Modifier.height(4.dp),
     playButtonSize: Dp = 32.dp,
-    borderColor: Color? = null
+    borderColor: Color? = null,
+    borderWidth: Dp = 3.dp,
 ) {
     val borderColor = borderColor ?: if (videoBlock.isCompleted()) {
         MaterialTheme.appColors.successGreen
@@ -732,7 +733,7 @@ fun CourseVideoItem(
         modifier = modifier
             .clip(MaterialTheme.appShapes.videoPreviewShape)
             .border(
-                width = 3.dp,
+                width = borderWidth,
                 color = borderColor,
                 shape = MaterialTheme.appShapes.videoPreviewShape
             )
@@ -783,7 +784,7 @@ fun CourseVideoItem(
                 style = titleStyle,
                 modifier = Modifier
                     .align(Alignment.TopStart),
-                maxLines = 2,
+                maxLines = 1,
                 overflow = TextOverflow.Ellipsis
             )
 

course/src/main/java/org/openedx/course/presentation/unit/container/CourseUnitContainerFragment.kt

@@ -612,19 +612,29 @@ class CourseUnitContainerFragment : Fragment(R.layout.fragment_course_unit_conta
                     } else {
                         null
                     }
+                    val borderWidth = if (isSelectedBlock) {
+                        3.dp
+                    } else {
+                        1.dp
+                    }
                     CourseVideoItem(
                         modifier = Modifier
                             .width(112.dp)
                             .height(63.dp),
                         videoBlock = block,
                         preview = videoPreview[block.id],
-                        progress = videoProgress[block.id] ?: 0f,
+                        progress = if (isSelectedBlock) {
+                            0f
+                        } else {
+                            videoProgress[block.id] ?: 0f
+                        },
                         onClick = {
                             onVideoClick(block)
                         },
                         titleStyle = MaterialTheme.appTypography.labelSmall,
                         playButtonSize = playButtonSize,
-                        borderColor = borderColor
+                        borderColor = borderColor,
+                        borderWidth = borderWidth
                     )
                 }
             }

course/src/main/java/org/openedx/course/presentation/unit/container/CourseUnitContainerViewModel.kt

@@ -93,7 +93,7 @@ class CourseUnitContainerViewModel(
     private val _currentBlock = MutableStateFlow<Block?>(null)
     val currentBlock = _currentBlock.asStateFlow()
 
-    private val _hierarchyPath = MutableStateFlow<String>("")
+    private val _hierarchyPath = MutableStateFlow("")
     val hierarchyPath = _hierarchyPath.asStateFlow()
 
     var nextButtonText = ""
@@ -422,12 +422,10 @@ class CourseUnitContainerViewModel(
     private fun buildHierarchyPath(block: Block): String {
         val pathComponents = mutableListOf<String>()
 
-        // Find the current block (Unit)
-        pathComponents.add(block.displayName)
-
-        // Find the parent Vertical block (but don't add it to path)
         val verticalBlock = findParentBlock(block.id)
         verticalBlock?.let { vertical ->
+            // Vertical name
+            pathComponents.add(0, vertical.displayName)
             // Find the parent Sequential block (Subsection)
             val sequentialBlock = findParentBlock(vertical.id)
             sequentialBlock?.let { sequential ->