Revert "Change to AnimatablePathView class from HasAnimatablePath pro…

…tocol🙇"
mono0926 · May 21, 2017 · aa30c5a · aa30c5a
1 parent f7937ac
commit aa30c5a
Show file tree

Hide file tree

Showing 6 changed files with 20 additions and 15 deletions.
diff --git a/NativePopup/Animatable/Animatable.swift b/NativePopup/Animatable/Animatable.swift
@@ -8,11 +8,11 @@
 
 import Foundation
 
-protocol Animatable {
+public protocol Animatable {
     func animate()
     var duration: TimeInterval { get }
 }
 
-extension Animatable {
+public extension Animatable {
     var duration: TimeInterval { return 0.3 }
 }
diff --git a/NativePopup/Animatable/AnimatableCrossView.swift b/NativePopup/Animatable/AnimatableCrossView.swift
@@ -8,9 +8,10 @@
 
 import Foundation
 
-class AnimatableCrossView: AnimatablePathView {
+class AnimatableCrossView: UIView, HasAnimatablePath {
+    let animatableLayer = CAShapeLayer()
     var duration: TimeInterval { return 0.4 }
-    override var animatablePath: UIBezierPath {
+    var path: UIBezierPath {
         let length = frame.width
         let path = UIBezierPath()
         path.move(to: CGPoint(x: length * 0.1, y: length * 0.1))

diff --git a/NativePopup/Animatable/AnimatableDoneView.swift b/NativePopup/Animatable/AnimatableDoneView.swift
@@ -8,8 +8,9 @@
 
 import Foundation
 
-class AnimatableDoneView: AnimatablePathView {
-    override var animatablePath: UIBezierPath {
+class AnimatableDoneView: UIView, HasAnimatablePath {
+    let animatableLayer = CAShapeLayer()
+    var path: UIBezierPath {
         let length = frame.width
         let path = UIBezierPath()
         path.move(to: CGPoint(x: length * 0.196, y: length * 0.527))

diff --git a/NativePopup/Animatable/HasAnimatableLayer.swift b/NativePopup/Animatable/HasAnimatableLayer.swift
@@ -8,12 +8,12 @@
 
 import Foundation
 
-protocol HasAnimatableLayer: Animatable {
+public protocol HasAnimatableLayer: Animatable {
     var animatableLayer: CAShapeLayer { get }
 }
 
-extension HasAnimatableLayer {
-    func animate() {
+public extension HasAnimatableLayer {
+    public func animate() {
         let animation = CABasicAnimation(keyPath: "strokeEnd")
         animation.duration = duration
         animation.fromValue = 0

diff --git a/NativePopup/Animatable/HasAnimatablePath.swift b/NativePopup/Animatable/HasAnimatablePath.swift
@@ -8,11 +8,14 @@
 
 import Foundation
 
-open class AnimatablePathView: UIView, HasAnimatableLayer {
-    let animatableLayer = CAShapeLayer()
-    open var animatablePath: UIBezierPath { fatalError("Should be overridden.") }
-    func setupLayer() {
-        animatableLayer.path = animatablePath.cgPath
+public protocol HasAnimatablePath: HasAnimatableLayer {
+    var path: UIBezierPath { get }
+    func setupLayer()
+}
+
+public extension HasAnimatablePath where Self: UIView {
+    public func setupLayer() {
+        animatableLayer.path = path.cgPath
         animatableLayer.fillColor = UIColor.clear.cgColor
         animatableLayer.strokeColor = tintColor.cgColor
         animatableLayer.lineWidth = 9

diff --git a/NativePopup/NativePopup.swift b/NativePopup/NativePopup.swift
@@ -110,7 +110,7 @@ public class NativePopup: UIView {
 
         [self, effectView, imageView, titleLabel, messageLabel].forEach { $0.translatesAutoresizingMaskIntoConstraints = false }
 
-        if let animatable = imageView as? AnimatablePathView {
+        if let animatable = imageView as? HasAnimatablePath {
             imageView.layoutIfNeeded()
             animatable.setupLayer()
         }