sam/paperscan
Public Access
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Releases 6 Wiki Activity

Fix crop corner handles positioned from center instead of top-left

Browse Source 
The crop container uses contentAlignment=Center, so the draggable corner
handles were offset from the middle of the box rather than the top-left,
leaving them far from the actual document corners. Anchor them with
align(TopStart) to match the Canvas coordinate space, and add a white
border for visibility.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
Sam Geyskens
2026-07-13 00:20:17 +02:00
parent e7ff34d3dd
commit c3158873ef
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
app/src/main/java/eu/geyskens/pdfscan/ui/screens/CropScreen.kt
View File

@@ -4,6 +4,7 @@ import android.graphics.Bitmap
import android.graphics.PointF import android.graphics.PointF
import androidx.compose.foundation.Image import androidx.compose.foundation.Image
import androidx.compose.foundation.background import androidx.compose.foundation.background
import androidx.compose.foundation.border
import androidx.compose.foundation.gestures.detectDragGestures import androidx.compose.foundation.gestures.detectDragGestures
import androidx.compose.foundation.layout.Arrangement import androidx.compose.foundation.layout.Arrangement
import androidx.compose.foundation.layout.Box import androidx.compose.foundation.layout.Box
@@ -104,12 +105,17 @@ fun CropScreen(vm: ScanViewModel, navController: NavController, pageId: String)
val handlePx = with(density) { 32.dp.toPx() } val handlePx = with(density) { 32.dp.toPx() }
Box( Box(
modifier = Modifier modifier = Modifier
// Position from the container's top-left, matching the Canvas
// coordinate space (the container uses contentAlignment = Center,
// which would otherwise offset handles from the middle).
.align(Alignment.TopStart)
.offset( .offset(
x = with(density) { (screen.x - handlePx / 2).toDp() }, x = with(density) { (screen.x - handlePx / 2).toDp() },
y = with(density) { (screen.y - handlePx / 2).toDp() }, y = with(density) { (screen.y - handlePx / 2).toDp() },
) )
.size(32.dp) .size(32.dp)
.background(Color(0xAA388BFD), CircleShape) .background(Color(0xAA388BFD), CircleShape)
.border(2.dp, Color.White, CircleShape)
.pointerInput(pageId, i, dispW, dispH) { .pointerInput(pageId, i, dispW, dispH) {
detectDragGestures { change, drag -> detectDragGestures { change, drag ->
change.consume() change.consume()