From 3a6aa7c0bb1ca0cca63bbe679ccdb5342c369871 Mon Sep 17 00:00:00 2001
From: Antoine Jaury <antoine@jaury.eu>
Date: Thu, 2 Jul 2026 17:46:39 +0200
Subject: [PATCH] new: add click-to-seek functionality

---
 .../fr/ajaury/gwenedeg/player/ui/PlayerScreen.kt |  4 +++-
 .../fr/ajaury/gwenedeg/player/ui/SubtitleList.kt | 16 ++++++++++++----
 2 files changed, 15 insertions(+), 5 deletions(-)

diff --git a/shared/src/commonMain/kotlin/fr/ajaury/gwenedeg/player/ui/PlayerScreen.kt b/shared/src/commonMain/kotlin/fr/ajaury/gwenedeg/player/ui/PlayerScreen.kt
index 7b66abf..90c6f99 100644
--- a/shared/src/commonMain/kotlin/fr/ajaury/gwenedeg/player/ui/PlayerScreen.kt
+++ b/shared/src/commonMain/kotlin/fr/ajaury/gwenedeg/player/ui/PlayerScreen.kt
@@ -128,7 +128,8 @@ fun PlayerScreen(
             modifier = Modifier
                 .padding(innerPadding)
                 .fillMaxSize()
-                .padding(16.dp),
+                .padding(vertical = 16.dp)
+                .padding(horizontal = 8.dp),
             horizontalAlignment = Alignment.CenterHorizontally,
         ) {
             SubtitleList(
@@ -142,6 +143,7 @@ fun PlayerScreen(
             )
 
             PlayerControl(
+                modifier = Modifier.padding(horizontal = 8.dp),
                 playbackState = uiState.playerState.playbackState,
                 playbackTiming = uiState.playerState.playbackTiming,
                 onSeek = onSeekToTimePart,
diff --git a/shared/src/commonMain/kotlin/fr/ajaury/gwenedeg/player/ui/SubtitleList.kt b/shared/src/commonMain/kotlin/fr/ajaury/gwenedeg/player/ui/SubtitleList.kt
index 819fbd8..3401f24 100644
--- a/shared/src/commonMain/kotlin/fr/ajaury/gwenedeg/player/ui/SubtitleList.kt
+++ b/shared/src/commonMain/kotlin/fr/ajaury/gwenedeg/player/ui/SubtitleList.kt
@@ -1,5 +1,6 @@
 package fr.ajaury.gwenedeg.player.ui
 
+import androidx.compose.foundation.clickable
 import androidx.compose.foundation.gestures.animateScrollBy
 import androidx.compose.foundation.interaction.collectIsDraggedAsState
 import androidx.compose.foundation.layout.Arrangement
@@ -8,10 +9,12 @@ import androidx.compose.foundation.layout.Column
 import androidx.compose.foundation.layout.PaddingValues
 import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.foundation.layout.fillMaxWidth
+import androidx.compose.foundation.layout.padding
 import androidx.compose.foundation.lazy.LazyColumn
 import androidx.compose.foundation.lazy.LazyListState
 import androidx.compose.foundation.lazy.itemsIndexed
 import androidx.compose.foundation.lazy.rememberLazyListState
+import androidx.compose.foundation.shape.RoundedCornerShape
 import androidx.compose.material3.MaterialTheme
 import androidx.compose.material3.Text
 import androidx.compose.runtime.Composable
@@ -23,6 +26,7 @@ import androidx.compose.runtime.remember
 import androidx.compose.runtime.setValue
 import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
+import androidx.compose.ui.draw.clip
 import androidx.compose.ui.graphics.Color
 import androidx.compose.ui.text.font.FontStyle
 import androidx.compose.ui.text.style.TextAlign
@@ -92,7 +96,7 @@ fun SubtitleList(
             state = listState,
             modifier = Modifier.fillMaxSize(),
             horizontalAlignment = Alignment.CenterHorizontally,
-            verticalArrangement = Arrangement.spacedBy(8.dp),
+            verticalArrangement = Arrangement.spacedBy(4.dp),
             // Half-height padding so the first and last lines can reach the vertical center.
             contentPadding = PaddingValues(vertical = maxHeight / 2),
         ) {
@@ -102,6 +106,7 @@ fun SubtitleList(
                     line = line,
                     isCurrent = isCurrent,
                     subtitlePreferences = subtitlePreferences,
+                    onClick = { onSeek(index) },
                 )
             }
         }
@@ -140,12 +145,17 @@ private fun Subtitle(
     line: SubtitleLine,
     isCurrent: Boolean,
     subtitlePreferences: SubtitlePreferences = SubtitlePreferences(),
+    onClick: () -> Unit = {},
 ) {
     val textAlpha = if (isCurrent) 1f else 0.35f
     val contentColor = MaterialTheme.colorScheme.onSurface.copy(alpha = textAlpha)
 
     Column(
-        modifier = Modifier.fillMaxWidth(),
+        modifier = Modifier
+            .clip(RoundedCornerShape(8.dp))
+            .clickable(onClick = onClick)
+            .padding(horizontal = 8.dp)
+            .padding(vertical = 4.dp),
         horizontalAlignment = Alignment.CenterHorizontally,
         verticalArrangement = Arrangement.spacedBy(2.dp),
     ) {
@@ -176,7 +186,6 @@ private fun TranscriptionLine(
         style = MaterialTheme.typography.bodyLarge.copy(fontSize = 20.sp),
         color = contentColor,
         textAlign = TextAlign.Center,
-        modifier = Modifier.fillMaxWidth(),
     )
 }
 
@@ -190,7 +199,6 @@ private fun TranslationLine(
         style = MaterialTheme.typography.bodySmall.copy(fontStyle = FontStyle.Italic),
         color = contentColor,
         textAlign = TextAlign.Center,
-        modifier = Modifier.fillMaxWidth(),
     )
 }