From bf734d5959a3e6d963114b4814736916fff03be2 Mon Sep 17 00:00:00 2001 From: Matthew Chen Date: Mon, 17 Dec 2018 14:17:40 -0500 Subject: [PATCH] Clean up ahead of PR. --- .../Views/ImageEditor/ImageEditorView.swift | 15 +++------------ 1 file changed, 3 insertions(+), 12 deletions(-) diff --git a/SignalMessaging/Views/ImageEditor/ImageEditorView.swift b/SignalMessaging/Views/ImageEditor/ImageEditorView.swift index cc6b9ab18..8455f2afb 100644 --- a/SignalMessaging/Views/ImageEditor/ImageEditorView.swift +++ b/SignalMessaging/Views/ImageEditor/ImageEditorView.swift @@ -37,7 +37,6 @@ public class ImageEditorView: UIView, ImageEditorModelDelegate { public func handleTouchGesture(_ gestureRecognizer: UIGestureRecognizer) { AssertIsOnMainThread() - let removeCurrentStroke = { if let stroke = self.currentStroke { self.model.remove(item: stroke) @@ -48,14 +47,16 @@ public class ImageEditorView: UIView, ImageEditorModelDelegate { let referenceView = self let unitSampleForGestureLocation = { () -> CGPoint in + // TODO: Smooth touch samples before converting into stroke samples. let location = gestureRecognizer.location(in: referenceView) let x = CGFloatClamp01(CGFloatInverseLerp(location.x, 0, referenceView.bounds.width)) let y = CGFloatClamp01(CGFloatInverseLerp(location.y, 0, referenceView.bounds.height)) return CGPoint(x: x, y: y) } - // TODO: + // TODO: Color picker. let strokeColor = UIColor.blue + // TODO: Tune stroke width. let unitStrokeWidth = ImageEditorStrokeItem.defaultUnitStrokeWidth() switch gestureRecognizer.state { @@ -237,16 +238,6 @@ public class ImageEditorView: UIView, ImageEditorModelDelegate { } lastForwardVector = forwardVector } - var hasSample = false - for unitSample in unitSamples { - let point = transformSampleToPoint(unitSample) - if hasSample { - bezierPath.addLine(to: point) - } else { - bezierPath.move(to: point) - hasSample = true - } - } shapeLayer.path = bezierPath.cgPath shapeLayer.fillColor = nil