From 2b25d875bcc5cd0313530391ff083352aa9bb9f2 Mon Sep 17 00:00:00 2001 From: Matthew Chen Date: Thu, 20 Dec 2018 09:29:15 -0500 Subject: [PATCH] Don't select a tool by default in image editor view. --- SignalMessaging/Views/ImageEditor/ImageEditorView.swift | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/SignalMessaging/Views/ImageEditor/ImageEditorView.swift b/SignalMessaging/Views/ImageEditor/ImageEditorView.swift index 2dfcd4268..67b7ea554 100644 --- a/SignalMessaging/Views/ImageEditor/ImageEditorView.swift +++ b/SignalMessaging/Views/ImageEditor/ImageEditorView.swift @@ -11,21 +11,26 @@ public class ImageEditorView: UIView, ImageEditorModelDelegate { private let model: ImageEditorModel enum EditorMode: String { + case none case brush case crop } - private var editorMode = EditorMode.brush { + private var editorMode = EditorMode.none { didSet { AssertIsOnMainThread() switch editorMode { + case .none: + editorGestureRecognizer?.isEnabled = false case .brush: // Brush strokes can start and end (and return from) outside the view. editorGestureRecognizer?.shouldAllowOutsideView = true + editorGestureRecognizer?.isEnabled = true case .crop: // Crop gestures can start and end (and return from) outside the view. editorGestureRecognizer?.shouldAllowOutsideView = true + editorGestureRecognizer?.isEnabled = true } } } @@ -209,6 +214,8 @@ public class ImageEditorView: UIView, ImageEditorModelDelegate { AssertIsOnMainThread() switch editorMode { + case .none: + break case .brush: handleBrushGesture(gestureRecognizer) case .crop: