Revert "Add CropAuxiliaryIndicatorConfig to allow custom CropAuxiliar…

…yIndicat…" (#404)

This reverts commit 44c11af.

Example/EmbeddedCropViewController.swift

@@ -68,7 +68,7 @@ class EmbeddedCropViewController: UIViewController {
             return
         }
 
-        cropViewController?.update(image.addFilter(filter: .mono))
+        cropViewController?.update(image.addFilter(filter: .Mono))
     }
 
     override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
@@ -185,28 +185,28 @@ extension EmbeddedCropViewController: CropViewControllerDelegate {
     }
 }
 
-enum FilterType: String {
-    case chrome = "CIPhotoEffectChrome"
-    case fade = "CIPhotoEffectFade"
-    case instant = "CIPhotoEffectInstant"
-    case mono = "CIPhotoEffectMono"
-    case noir = "CIPhotoEffectNoir"
-    case process = "CIPhotoEffectProcess"
-    case tonal = "CIPhotoEffectTonal"
-    case transfer = "CIPhotoEffectTransfer"
+enum FilterType : String {
+    case Chrome = "CIPhotoEffectChrome"
+    case Fade = "CIPhotoEffectFade"
+    case Instant = "CIPhotoEffectInstant"
+    case Mono = "CIPhotoEffectMono"
+    case Noir = "CIPhotoEffectNoir"
+    case Process = "CIPhotoEffectProcess"
+    case Tonal = "CIPhotoEffectTonal"
+    case Transfer = "CIPhotoEffectTransfer"
 }
 
 extension UIImage {
-    func addFilter(filter: FilterType) -> UIImage {
+    func addFilter(filter : FilterType) -> UIImage {
         let filter = CIFilter(name: filter.rawValue)
-        // Convert UIImage to CIImage and set as input
+        // convert UIImage to CIImage and set as input
         let ciInput = CIImage(image: self)
         filter?.setValue(ciInput, forKey: "inputImage")
-        // Get output CIImage, render as CGImage first to retain proper UIImage scale
+        // get output CIImage, render as CGImage first to retain proper UIImage scale
         let ciOutput = filter?.outputImage
         let ciContext = CIContext()
         let cgImage = ciContext.createCGImage(ciOutput!, from: (ciOutput?.extent)!)
-        // Return the image
+        //Return the image
         return UIImage(cgImage: cgImage!)
     }
 }

Example/zh-Hans.lproj/Main.storyboard

@@ -1,9 +1,9 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<document type="com.apple.InterfaceBuilder3.CocoaTouch.Storyboard.XIB" version="3.0" toolsVersion="32700.99.1234" targetRuntime="iOS.CocoaTouch" propertyAccessControl="none" useAutolayout="YES" useTraitCollections="YES" useSafeAreas="YES" colorMatched="YES" initialViewController="BYZ-38-t0r">
+<document type="com.apple.InterfaceBuilder3.CocoaTouch.Storyboard.XIB" version="3.0" toolsVersion="22505" targetRuntime="iOS.CocoaTouch" propertyAccessControl="none" useAutolayout="YES" useTraitCollections="YES" useSafeAreas="YES" colorMatched="YES" initialViewController="BYZ-38-t0r">
     <device id="ipad7_9" orientation="portrait" layout="fullscreen" appearance="light"/>
     <dependencies>
         <deployment identifier="iOS"/>
-        <plugIn identifier="com.apple.InterfaceBuilder.IBCocoaTouchPlugin" version="22684"/>
+        <plugIn identifier="com.apple.InterfaceBuilder.IBCocoaTouchPlugin" version="22504"/>
         <capability name="Safe area layout guides" minToolsVersion="9.0"/>
         <capability name="documents saved in the Xcode 8 format" minToolsVersion="8.0"/>
     </dependencies>
@@ -181,6 +181,21 @@
             </objects>
             <point key="canvasLocation" x="1117.5999999999999" y="5.8470764617691158"/>
         </scene>
+        <!--Crop View Controller-->
+        <scene sceneID="djG-LZ-hRH">
+            <objects>
+                <viewController id="xJq-Ev-XcJ" customClass="CropViewController" customModule="Mantis" sceneMemberID="viewController">
+                    <view key="view" contentMode="scaleToFill" id="nLp-HP-0xC">
+                        <rect key="frame" x="0.0" y="0.0" width="744" height="1133"/>
+                        <autoresizingMask key="autoresizingMask" widthSizable="YES" heightSizable="YES"/>
+                        <viewLayoutGuide key="safeArea" id="0JZ-7h-zil"/>
+                        <color key="backgroundColor" white="1" alpha="1" colorSpace="custom" customColorSpace="genericGamma22GrayColorSpace"/>
+                    </view>
+                </viewController>
+                <placeholder placeholderIdentifier="IBFirstResponder" id="TRU-F3-FYg" userLabel="First Responder" sceneMemberID="firstResponder"/>
+            </objects>
+            <point key="canvasLocation" x="1042" y="819"/>
+        </scene>
     </scenes>
     <resources>
         <image name="sunflower" width="3648" height="5472"/>

Mantis.xcodeproj/project.pbxproj

@@ -75,7 +75,6 @@
 		F24DCFE829AB0A7000D8E8C1 /* CropAuxiliaryIndicatorViewTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = F24DCFE729AB0A7000D8E8C1 /* CropAuxiliaryIndicatorViewTests.swift */; };
 		F24DCFEC29AF2A6000D8E8C1 /* CropAuxiliaryIndicatorView+Accessibility.swift in Sources */ = {isa = PBXBuildFile; fileRef = F24DCFEB29AF2A6000D8E8C1 /* CropAuxiliaryIndicatorView+Accessibility.swift */; };
 		F251B36E299B9A52006325B0 /* CropWorkbenchViewTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = F251B36D299B9A52006325B0 /* CropWorkbenchViewTests.swift */; };
-		F28FF64A2C4A492A002F5F06 /* CropAuxiliaryIndicatorConfig.swift in Sources */ = {isa = PBXBuildFile; fileRef = F28FF6492C4A492A002F5F06 /* CropAuxiliaryIndicatorConfig.swift */; };
 		F2AD53412A46DB8B00AE9C87 /* ImageAutoAdjustHelper.swift in Sources */ = {isa = PBXBuildFile; fileRef = F2AD53402A46DB8B00AE9C87 /* ImageAutoAdjustHelper.swift */; };
 		F2B4FDA42A3B661B00667F22 /* SlideDial.swift in Sources */ = {isa = PBXBuildFile; fileRef = F2B4FDA32A3B661B00667F22 /* SlideDial.swift */; };
 		F2B4FDA72A3B69B600667F22 /* SlideRulerPositionHelper.swift in Sources */ = {isa = PBXBuildFile; fileRef = F2B4FDA52A3B69B600667F22 /* SlideRulerPositionHelper.swift */; };
@@ -212,7 +211,6 @@
 		F24DCFE729AB0A7000D8E8C1 /* CropAuxiliaryIndicatorViewTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = CropAuxiliaryIndicatorViewTests.swift; sourceTree = "<group>"; };
 		F24DCFEB29AF2A6000D8E8C1 /* CropAuxiliaryIndicatorView+Accessibility.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; path = "CropAuxiliaryIndicatorView+Accessibility.swift"; sourceTree = "<group>"; };
 		F251B36D299B9A52006325B0 /* CropWorkbenchViewTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = CropWorkbenchViewTests.swift; sourceTree = "<group>"; };
-		F28FF6492C4A492A002F5F06 /* CropAuxiliaryIndicatorConfig.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = CropAuxiliaryIndicatorConfig.swift; sourceTree = "<group>"; };
 		F2AD53402A46DB8B00AE9C87 /* ImageAutoAdjustHelper.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; path = ImageAutoAdjustHelper.swift; sourceTree = "<group>"; };
 		F2B4FDA32A3B661B00667F22 /* SlideDial.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = SlideDial.swift; sourceTree = "<group>"; };
 		F2B4FDA52A3B69B600667F22 /* SlideRulerPositionHelper.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; path = SlideRulerPositionHelper.swift; sourceTree = "<group>"; };
@@ -509,7 +507,6 @@
 				OBJ_36 /* Mantis.swift */,
 				6659429C2877DAE80096A5DB /* Config.swift */,
 				663FCF3728866DDA003D3B9E /* CropViewConfig.swift */,
-				F28FF6492C4A492A002F5F06 /* CropAuxiliaryIndicatorConfig.swift */,
 				663FCF3828866DDA003D3B9E /* CropToolbarConfig.swift */,
 				6659429E2877DC8A0096A5DB /* Enum.swift */,
 				665942A02877DD1C0096A5DB /* CropData.swift */,
@@ -798,7 +795,6 @@
 				FEDAAD8625205CC300D95667 /* RatioSelector.swift in Sources */,
 				OBJ_93 /* CropVisualEffectView.swift in Sources */,
 				269AF94227437EE400F7FAF6 /* RotationDialConfig.swift in Sources */,
-				F28FF64A2C4A492A002F5F06 /* CropAuxiliaryIndicatorConfig.swift in Sources */,
 				OBJ_94 /* RatioOptions.swift in Sources */,
 				OBJ_95 /* Angle.swift in Sources */,
 				OBJ_97 /* RotationCalculator.swift in Sources */,

Sources/Mantis/CropView/CropAuxiliaryIndicatorView.swift

@@ -9,10 +9,8 @@
 import UIKit
 
 final class CropAuxiliaryIndicatorView: UIView, CropAuxiliaryIndicatorViewProtocol {
-    private var borderNormalColor = UIColor.white
-    private var borderHintColor = UIColor.white
-    private var cornerHandleColor = UIColor.white
-    private var edgeLineHandleColor = UIColor.white
+    private var boarderNormalColor = UIColor.white
+    private var boarderHintColor = UIColor.white
     private let cornerHandleLength = CGFloat(20.0)
     private let edgeLineHandleLength = CGFloat(30.0)
     private let handleThickness = CGFloat(3.0)
@@ -51,22 +49,16 @@ final class CropAuxiliaryIndicatorView: UIView, CropAuxiliaryIndicatorViewProtoc
         }
     }
 
-    init(frame: CGRect, config: CropAuxiliaryIndicatorConfig = CropAuxiliaryIndicatorConfig()) {
+    init(frame: CGRect, 
+         cropBoxHotAreaUnit: CGFloat,
+         disableCropBoxDeformation: Bool = false,
+         style: CropAuxiliaryIndicatorStyleType = .normal) {
         super.init(frame: frame)
         clipsToBounds = false
         backgroundColor = .clear
-
-        cropBoxHotAreaUnit = config.cropBoxHotAreaUnit
-        disableCropBoxDeformation = config.disableCropBoxDeformation
-        style = config.style
-
-        borderNormalColor = config.borderNormalColor
-        borderHintColor = config.borderHintColor
-        cornerHandleColor = config.cornerHandleColor
-        edgeLineHandleColor = config.edgeLineHandleColor
-        gridMainColor = config.gridMainColor
-        gridSecondaryColor = config.gridSecondaryColor
-
+        self.cropBoxHotAreaUnit = cropBoxHotAreaUnit
+        self.disableCropBoxDeformation = disableCropBoxDeformation
+        self.style = style
         setup()
     }
 
@@ -75,18 +67,15 @@ final class CropAuxiliaryIndicatorView: UIView, CropAuxiliaryIndicatorViewProtoc
         backgroundColor = .clear
     }
 
-    private func createNewLine(withNormalColor normalColor: UIColor = .white) -> UIView {
+    private func createNewLine() -> UIView {
         let view = UIView()
         view.frame = .zero
-
         if style == .normal {
-            view.backgroundColor = normalColor
+            view.backgroundColor = .white
         } else {
             view.backgroundColor = .clear
         }
-
         addSubview(view)
-
         return view
     }
 
@@ -96,19 +85,19 @@ final class CropAuxiliaryIndicatorView: UIView, CropAuxiliaryIndicatorViewProtoc
         borderLine.layer.borderWidth = borderThickness
 
         if style == .normal {
-            borderLine.layer.borderColor = borderNormalColor.cgColor
-            hintLine.backgroundColor = borderHintColor
+            borderLine.layer.borderColor = boarderNormalColor.cgColor
+            hintLine.backgroundColor = boarderHintColor
         } else {
             borderLine.layer.borderColor = UIColor.clear.cgColor
             hintLine.backgroundColor = .clear
         }
 
         for _ in 0..<8 {
-            cornerHandles.append(createNewLine(withNormalColor: cornerHandleColor))
+            cornerHandles.append(createNewLine())
         }
 
         for _ in 0..<4 {
-            edgeLineHandles.append(createNewLine(withNormalColor: edgeLineHandleColor))
+            edgeLineHandles.append(createNewLine())
         }
 
         setupAccessibilityHelperViews()
@@ -188,7 +177,7 @@ final class CropAuxiliaryIndicatorView: UIView, CropAuxiliaryIndicatorViewProtoc
         borderLine.layer.backgroundColor = UIColor.clear.cgColor
         borderLine.layer.borderWidth = borderThickness
         if style == .normal {
-            borderLine.layer.borderColor = borderNormalColor.cgColor
+            borderLine.layer.borderColor = boarderNormalColor.cgColor
         } else {
             borderLine.layer.borderColor = UIColor.clear.cgColor
         }

Sources/Mantis/CropView/CropView+Touches.swift

@@ -17,7 +17,7 @@ extension CropView {
             return rotationControlView.getTouchTarget(with: pointInRotationControlView)
         }
 
-        if !cropViewConfig.cropAuxiliaryIndicatorConfig.disableCropBoxDeformation && isHitGridOverlayView(by: newPoint) {
+        if !cropViewConfig.disableCropBoxDeformation && isHitGridOverlayView(by: newPoint) {
             return self
         }
 
@@ -29,7 +29,7 @@ extension CropView {
     }
 
     private func isHitGridOverlayView(by touchPoint: CGPoint) -> Bool {
-        let hotAreaUnit = cropViewConfig.cropAuxiliaryIndicatorConfig.cropBoxHotAreaUnit
+        let hotAreaUnit = cropViewConfig.cropBoxHotAreaUnit
 
         return cropAuxiliaryIndicatorView.frame.insetBy(dx: -hotAreaUnit/2, dy: -hotAreaUnit/2).contains(touchPoint)
         && !cropAuxiliaryIndicatorView.frame.insetBy(dx: hotAreaUnit/2, dy: hotAreaUnit/2).contains(touchPoint)

Sources/Mantis/CropView/CropView+UIScrollViewDelegate.swift

@@ -25,7 +25,7 @@ extension CropView: UIScrollViewDelegate {
     }
 
     func scrollViewDidZoom(_ scrollView: UIScrollView) {
-        guard !scrollView.subviews.isEmpty else {
+        guard scrollView.subviews.count > 0 else {
             return
         }
 
@@ -34,7 +34,7 @@ extension CropView: UIScrollViewDelegate {
         let offsetX: CGFloat = max((scrollView.bounds.size.width - scrollView.contentSize.width) * 0.5, 0.0)
         let offsetY: CGFloat = max((scrollView.bounds.size.height - scrollView.contentSize.height) * 0.5, 0.0)
 
-        subView.center = CGPoint(x: scrollView.contentSize.width * 0.5 + offsetX, y: scrollView.contentSize.height * 0.5 + offsetY)
+        subView.center = CGPointMake(scrollView.contentSize.width * 0.5 + offsetX, scrollView.contentSize.height * 0.5 + offsetY)
     }
 
     func scrollViewDidEndDecelerating(_ scrollView: UIScrollView) {

Sources/Mantis/CropViewConfig.swift

@@ -4,38 +4,12 @@ public struct CropViewConfig {
     /**
         This value is for how easy to drag crop box. The bigger, the easier.
      */
-    @available(*, deprecated, message: "Use cropAuxiliaryIndicatorStyle.cropBoxHotAreaUnit instead")
-    public var cropBoxHotAreaUnit: CGFloat {
-        get {
-            cropAuxiliaryIndicatorConfig.cropBoxHotAreaUnit
-        }
-        set {
-            cropAuxiliaryIndicatorConfig.cropBoxHotAreaUnit = newValue
-        }
-    }
-
-    @available(*, deprecated, message: "Use cropAuxiliaryIndicatorStyle.disableCropBoxDeformation instead")
-    public var disableCropBoxDeformation: Bool {
-        get {
-            cropAuxiliaryIndicatorConfig.disableCropBoxDeformation
-        }
-        set {
-            cropAuxiliaryIndicatorConfig.disableCropBoxDeformation = newValue
-        }
-    }
-
-    @available(*, deprecated, message: "Use cropAuxiliaryIndicatorConfig.style instead")
-    public var cropAuxiliaryIndicatorStyle: CropAuxiliaryIndicatorStyleType {
-        get {
-            cropAuxiliaryIndicatorConfig.style
-        }
-        set {
-            cropAuxiliaryIndicatorConfig.style = newValue
+    public var cropBoxHotAreaUnit: CGFloat = 32 {
+        didSet {
+            assert(cropBoxHotAreaUnit > 0)
         }
     }
 
-    public var cropAuxiliaryIndicatorConfig = CropAuxiliaryIndicatorConfig()
-
     public var cropShapeType: CropShapeType = .rect
 
     public var cropBorderWidth: CGFloat = 0 {
@@ -57,6 +31,8 @@ public struct CropViewConfig {
 
     public var cropMaskVisualEffectType: CropMaskVisualEffectType = .blurDark
 
+    public var cropAuxiliaryIndicatorStyle: CropAuxiliaryIndicatorStyleType = .normal
+
     public var presetTransformationType: PresetTransformationType = .none
 
     public var minimumZoomScale: CGFloat = 1 {
@@ -90,11 +66,9 @@ public struct CropViewConfig {
 
     public var rotateCropBoxFor90DegreeRotation = true
 
-    public var minimumCropBoxSize: CGFloat = 42 {
-        didSet {
-            assert(minimumCropBoxSize >= 4)
-        }
-    }
+    var minimumCropBoxSize: CGFloat = 42
+
+    public var disableCropBoxDeformation = false
 
     public enum BuiltInRotationControlViewType {
         case rotationDial(config: RotationDialConfig = .init())

Sources/Mantis/Mantis.swift

@@ -97,7 +97,9 @@ private func buildCropView(withImage image: UIImage,
                            config cropViewConfig: CropViewConfig,
                            rotationControlView: RotationControlViewProtocol?) -> CropViewProtocol {
     let cropAuxiliaryIndicatorView = CropAuxiliaryIndicatorView(frame: .zero,
-                                                                config: cropViewConfig.cropAuxiliaryIndicatorConfig)
+                                                                cropBoxHotAreaUnit: cropViewConfig.cropBoxHotAreaUnit,
+                                                                disableCropBoxDeformation: cropViewConfig.disableCropBoxDeformation,
+                                                                style: cropViewConfig.cropAuxiliaryIndicatorStyle)
     let imageContainer = ImageContainer(image: image)
     let cropView = CropView(image: image,
                             cropViewConfig: cropViewConfig,
@@ -114,7 +116,7 @@ private func buildCropView(withImage image: UIImage,
 private func buildCropViewModel(with cropViewConfig: CropViewConfig) -> CropViewModelProtocol {
     CropViewModel(
         cropViewPadding: cropViewConfig.padding,
-        hotAreaUnit: cropViewConfig.cropAuxiliaryIndicatorConfig.cropBoxHotAreaUnit
+        hotAreaUnit: cropViewConfig.cropBoxHotAreaUnit
     )
 }
 

Sources/Mantis/MaskBackground/CropMaskProtocal.swift

@@ -8,7 +8,7 @@
 
 import UIKit
 
-private let minOverLayerUnit: CGFloat = 4
+private let minOverLayerUnit: CGFloat = 30
 private let initialFrameLength: CGFloat = 1000
 
 protocol CropMaskProtocol: UIView {