From 24bac8c8ed80b2dde6426a31b9dcbd792a145bc0 Mon Sep 17 00:00:00 2001 From: Alex Hart Date: Tue, 1 Sep 2026 15:29:44 -0300 Subject: [PATCH] Don't allow dots for crop. --- .../org/signal/mediasend/screens/edit/ImageController.kt | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/feature/media-send/src/main/java/org/signal/mediasend/screens/edit/ImageController.kt b/feature/media-send/src/main/java/org/signal/mediasend/screens/edit/ImageController.kt index fcd7c3bc19..ff9725a2d0 100644 --- a/feature/media-send/src/main/java/org/signal/mediasend/screens/edit/ImageController.kt +++ b/feature/media-send/src/main/java/org/signal/mediasend/screens/edit/ImageController.kt @@ -475,6 +475,7 @@ internal class ImageController( } fun enterCropMode() { + clearSelection() editorModel.startCrop() initialDialScale = editorModel.mainImage?.localScaleX ?: 1f transitionTo(Mode.CROP) @@ -559,7 +560,9 @@ internal class ImageController( } fun onEntityDown(element: EditorElement?) { - if (mode == Mode.ZOOM) { + // Crop makes the main image selectable so it can be panned inside the frame, but selecting it would draw the + // selection guide around the whole image, well outside the crop frame. + if (mode == Mode.ZOOM || mode == Mode.CROP) { return }