Merge pull request #295 from CookieySun/feature/entire_card_tappable

Change transition area to cover entire card
DroidKaigi · Aug 11, 2024 · 911e8b8 · 911e8b8
2 parents 9b090cf + e2adfaf
commit 911e8b8
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/...ui/src/commonMain/kotlin/io/github/droidkaigi/confsched/ui/component/TimetableItemCard.kt b/...ui/src/commonMain/kotlin/io/github/droidkaigi/confsched/ui/component/TimetableItemCard.kt
@@ -52,10 +52,12 @@ fun TimetableItemCard(
     ProvideRoomTheme(timetableItem.room.getThemeKey()) {
         Column(
             modifier = modifier
+                .testTag(TimetableItemCardTestTag)
                 .border(
                     border = BorderStroke(width = 1.dp, color = MaterialTheme.colorScheme.outlineVariant),
                     shape = RoundedCornerShape(5.dp),
                 )
+                .clickable { onTimetableItemClick(timetableItem) }
                 .padding(15.dp),
         ) {
             Box {
@@ -87,8 +89,7 @@ fun TimetableItemCard(
                 fontSize = 24.sp,
                 modifier = Modifier
                     .testTag(TimetableItemCardTestTag)
-                    .padding(bottom = 5.dp)
-                    .clickable { onTimetableItemClick(timetableItem) },
+                    .padding(bottom = 5.dp),
             )
             timetableItem.speakers.forEach { speaker ->
                 Row {