From 2877563336471d10c43174e29b55c6cfad3f8571 Mon Sep 17 00:00:00 2001
From: Chris Arriola <carriola@google.com>
Date: Thu, 25 Apr 2024 16:34:35 -0700
Subject: [PATCH] [Jetcaster] Podcast details - adjusting spacing and show
 episode summary.

---
 .../ui/podcast/PodcastDetailsScreen.kt         |  3 ++-
 .../jetcaster/ui/shared/EpisodeListItem.kt     | 18 +++++++++++-------
 2 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/Jetcaster/app/src/main/java/com/example/jetcaster/ui/podcast/PodcastDetailsScreen.kt b/Jetcaster/app/src/main/java/com/example/jetcaster/ui/podcast/PodcastDetailsScreen.kt
index 8cc1200881..7bc8d1bad8 100644
--- a/Jetcaster/app/src/main/java/com/example/jetcaster/ui/podcast/PodcastDetailsScreen.kt
+++ b/Jetcaster/app/src/main/java/com/example/jetcaster/ui/podcast/PodcastDetailsScreen.kt
@@ -181,7 +181,8 @@ fun PodcastDetailsContent(
                 onClick = navigateToPlayer,
                 onQueueEpisode = onQueueEpisode,
                 modifier = Modifier.fillMaxWidth(),
-                showPodcastImage = false
+                showPodcastImage = false,
+                showSummary = true
             )
         }
     }
diff --git a/Jetcaster/app/src/main/java/com/example/jetcaster/ui/shared/EpisodeListItem.kt b/Jetcaster/app/src/main/java/com/example/jetcaster/ui/shared/EpisodeListItem.kt
index bafb863074..b31f1ca472 100644
--- a/Jetcaster/app/src/main/java/com/example/jetcaster/ui/shared/EpisodeListItem.kt
+++ b/Jetcaster/app/src/main/java/com/example/jetcaster/ui/shared/EpisodeListItem.kt
@@ -72,6 +72,7 @@ fun EpisodeListItem(
     onQueueEpisode: (PlayerEpisode) -> Unit,
     modifier: Modifier = Modifier,
     showPodcastImage: Boolean = true,
+    showSummary: Boolean = false,
 ) {
     Box(modifier = modifier.padding(vertical = 8.dp, horizontal = 16.dp)) {
         Surface(
@@ -90,7 +91,8 @@ fun EpisodeListItem(
                     episode = episode,
                     podcast = podcast,
                     showPodcastImage = showPodcastImage,
-                    modifier = Modifier.padding(bottom = 4.dp)
+                    showSummary = showSummary,
+                    modifier = Modifier.padding(bottom = 8.dp)
                 )
 
                 // Bottom Part
@@ -183,10 +185,11 @@ private fun EpisodeListItemFooter(
 }
 
 @Composable
-fun EpisodeListItemHeader(
+private fun EpisodeListItemHeader(
     episode: EpisodeInfo,
     podcast: PodcastInfo,
     showPodcastImage: Boolean,
+    showSummary: Boolean,
     modifier: Modifier = Modifier
 ) {
     Row(modifier = modifier) {
@@ -199,16 +202,16 @@ fun EpisodeListItemHeader(
             Text(
                 text = episode.title,
                 maxLines = 2,
-                minLines = 2,
+                minLines = 1,
                 overflow = TextOverflow.Ellipsis,
                 style = MaterialTheme.typography.titleMedium,
-                modifier = Modifier.padding(vertical = 8.dp)
+                modifier = Modifier.padding(vertical = 2.dp)
             )
 
             Text(
-                text = podcast.title,
+                text = if (showSummary) episode.summary else podcast.title,
                 maxLines = 2,
-                minLines = 2,
+                minLines = 1,
                 overflow = TextOverflow.Ellipsis,
                 style = MaterialTheme.typography.titleSmall,
             )
@@ -261,7 +264,8 @@ private fun EpisodeListItemPreview() {
             episode = PreviewEpisodes[0],
             podcast = PreviewPodcasts[0],
             onClick = {},
-            onQueueEpisode = {}
+            onQueueEpisode = {},
+            showSummary = true
         )
     }
 }