From 8cd97fc928bd9e51e2f54257716e089e99779982 Mon Sep 17 00:00:00 2001 From: Barnabas Balogh Date: Mon, 26 Jan 2026 20:32:41 +0100 Subject: [PATCH] refactor: rename player gesture handlers for clarity - Rename `PlayerGesturesLayer` parameters to be more descriptive of the gesture type (e.g., `onDoubleTapLeft`, `onDoubleTapCenter`, `onDoubleTapRight`). - Update the `PlayerScreen` to use the new, more specific gesture handler names. --- .../java/hu/bbara/purefin/player/ui/PlayerScreen.kt | 6 +++--- .../player/ui/components/PlayerGesturesLayer.kt | 13 +++++++------ 2 files changed, 10 insertions(+), 9 deletions(-) diff --git a/app/src/main/java/hu/bbara/purefin/player/ui/PlayerScreen.kt b/app/src/main/java/hu/bbara/purefin/player/ui/PlayerScreen.kt index 0d5e02d..4d2b521 100644 --- a/app/src/main/java/hu/bbara/purefin/player/ui/PlayerScreen.kt +++ b/app/src/main/java/hu/bbara/purefin/player/ui/PlayerScreen.kt @@ -89,9 +89,9 @@ fun PlayerScreen( PlayerGesturesLayer( modifier = Modifier.fillMaxSize(), onTap = { viewModel.toggleControlsVisibility() }, - onSeekForward = { viewModel.seekBy(30_000) }, - onSeekBackward = { viewModel.seekBy(-10_000) }, - onResumePause = {viewModel.togglePlayPause()}, + onDoubleTapRight = { viewModel.seekBy(30_000) }, + onDoubleTapLeft = { viewModel.seekBy(-10_000) }, + onDoubleTapCenter = {viewModel.togglePlayPause()}, onVerticalDragLeft = { delta -> val diff = (-delta / 800f) brightness = (brightness + diff).coerceIn(0f, 1f) diff --git a/app/src/main/java/hu/bbara/purefin/player/ui/components/PlayerGesturesLayer.kt b/app/src/main/java/hu/bbara/purefin/player/ui/components/PlayerGesturesLayer.kt index c77ee55..d334e3c 100644 --- a/app/src/main/java/hu/bbara/purefin/player/ui/components/PlayerGesturesLayer.kt +++ b/app/src/main/java/hu/bbara/purefin/player/ui/components/PlayerGesturesLayer.kt @@ -1,6 +1,7 @@ package hu.bbara.purefin.player.ui.components import androidx.compose.foundation.gestures.detectDragGestures +import androidx.compose.foundation.gestures.detectHorizontalDragGestures import androidx.compose.foundation.gestures.detectTapGestures import androidx.compose.foundation.layout.Box import androidx.compose.foundation.layout.fillMaxSize @@ -12,9 +13,9 @@ import androidx.compose.ui.input.pointer.pointerInput fun PlayerGesturesLayer( modifier: Modifier = Modifier, onTap: () -> Unit, - onResumePause: () -> Unit, - onSeekForward: () -> Unit, - onSeekBackward: () -> Unit, + onDoubleTapCenter: () -> Unit, + onDoubleTapRight: () -> Unit, + onDoubleTapLeft: () -> Unit, onVerticalDragLeft: (delta: Float) -> Unit, onVerticalDragRight: (delta: Float) -> Unit ) { @@ -30,11 +31,11 @@ fun PlayerGesturesLayer( val oneThird = screenWidth / 3 val secondThird = oneThird * 2 if (offset.x < oneThird) { - onSeekBackward() + onDoubleTapLeft() } else if (offset.x >= oneThird && offset.x <= secondThird) { - onResumePause() + onDoubleTapCenter() } else { - onSeekForward() + onDoubleTapRight() } } )