Update list spacing in the schedule, add separator item between days

Author: zsmb13

shared/src/commonMain/kotlin/org/jetbrains/kotlinconf/screens/ScheduleScreen.kt

@@ -8,8 +8,10 @@ import androidx.compose.foundation.layout.Box
 import androidx.compose.foundation.layout.Column
 import androidx.compose.foundation.layout.PaddingValues
 import androidx.compose.foundation.layout.RowScope
+import androidx.compose.foundation.layout.Spacer
 import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.foundation.layout.fillMaxWidth
+import androidx.compose.foundation.layout.height
 import androidx.compose.foundation.layout.padding
 import androidx.compose.foundation.lazy.LazyColumn
 import androidx.compose.foundation.lazy.LazyListState
@@ -441,6 +443,7 @@ private fun ScheduleList(
             scheduleItems,
             key = {
                 when (it) {
+                    is DaySeparatorItem -> it.id
                     is DayHeaderItem -> it.value.date.toString()
                     is ServiceEventGroupItem -> it.value.map { it.id.id }
                     is ServiceEventItem -> it.value.id.id
@@ -453,6 +456,10 @@ private fun ScheduleList(
         ) { item ->
             Box(Modifier.animateItem()) {
                 when (item) {
+                    is DaySeparatorItem -> {
+                        Spacer(modifier = Modifier.height(48.dp))
+                    }
+
                     is DayHeaderItem -> {
                         val date = item.value.date
                         val dayValues = DayValues.map[date]
@@ -463,7 +470,7 @@ private fun ScheduleList(
                             line2 = dayValues?.line2 ?: "",
                             modifier = Modifier
                                 .fillMaxWidth()
-                                .padding(top = 8.dp)
+                                .padding(vertical = 16.dp)
                                 .semantics { heading() }
                         )
                     }
@@ -475,9 +482,7 @@ private fun ScheduleList(
                             modifier = Modifier
                                 .padding(horizontal = 12.dp)
                                 .padding(top = 24.dp, bottom = 8.dp)
-                                .semantics {
-                                    heading()
-                                }
+                                .semantics { heading() }
                         )
                     }
 

shared/src/commonMain/kotlin/org/jetbrains/kotlinconf/screens/ScheduleViewModel.kt

@@ -18,6 +18,7 @@ import org.jetbrains.kotlinconf.Day
 import org.jetbrains.kotlinconf.Score
 import org.jetbrains.kotlinconf.SessionCardView
 import org.jetbrains.kotlinconf.SessionId
+import org.jetbrains.kotlinconf.SessionState
 import org.jetbrains.kotlinconf.TagValues
 import org.jetbrains.kotlinconf.TimeProvider
 import org.jetbrains.kotlinconf.TimeSlot
@@ -30,6 +31,8 @@ import org.jetbrains.kotlinconf.utils.removeDiacritics
 
 sealed interface ScheduleListItem
 
+data class DaySeparatorItem(val id: String) : ScheduleListItem
+
 data class DayHeaderItem(val value: Day) : ScheduleListItem
 
 data class TimeSlotTitleItem(val value: TimeSlot) : ScheduleListItem
@@ -176,7 +179,7 @@ class ScheduleViewModel(
         days: List<Day>,
         searchQuery: String,
         selectedTags: List<String>,
-    ): List<ScheduleListItem> = buildList {
+    ): List<SessionItem> = buildList {
         for (day in days) {
             for (timeSlot in day.timeSlots) {
                 for (session in timeSlot.sessions) {
@@ -211,7 +214,10 @@ class ScheduleViewModel(
         var seenPastSlot = false
 
         val items = buildList {
-            days.forEach { day ->
+            days.forEachIndexed { index, day ->
+                if (index > 0) {
+                    add(DaySeparatorItem("day-separator-$index"))
+                }
                 add(DayHeaderItem(day))
 
                 day.timeSlots.forEachIndexed { index, timeSlot ->