apply review comments

deliveryhero · Jun 17, 2020 · 6b11ef9 · 6b11ef9
1 parent 1f7ac01
commit 6b11ef9
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/Sources/ParallaxPagerView.swift b/Sources/ParallaxPagerView.swift
@@ -34,7 +34,7 @@ public final class ParallaxPagerView: UIView {
   private var contentOffsetObservation: NSKeyValueObservation?
   private var contentInsetObservation: NSKeyValueObservation?
 
-  private unowned let containerViewController: UIViewController
+  private weak var containerViewController: UIViewController?
 
   private var headerHeightConstraint: NSLayoutConstraint?
   private var contentViewLeadingConstraint: NSLayoutConstraint?
@@ -224,8 +224,8 @@ public final class ParallaxPagerView: UIView {
 
     layoutInternalScrollView()
 
-    containerViewController.automaticallyAdjustsScrollViewInsets = false
-    containerViewController.extendedLayoutIncludesOpaqueBars = false
+    containerViewController?.automaticallyAdjustsScrollViewInsets = false
+    containerViewController?.extendedLayoutIncludesOpaqueBars = false
     preservesSuperviewLayoutMargins = true
 
     headerView.clipsToBounds = true
@@ -505,7 +505,7 @@ public final class ParallaxPagerView: UIView {
 
     vc.willMove(toParent: containerViewController)
     internalScrollView.insertSubview(vc.view, at: 0)
-    containerViewController.addChild(vc)
+    containerViewController?.addChild(vc)
     vc.didMove(toParent: containerViewController)
 
     contentViewTrailingConstraint = NSLayoutConstraint(

diff --git a/Sources/Protocols.swift b/Sources/Protocols.swift
@@ -15,7 +15,7 @@ public protocol ParallaxContentViewController {
   func scrollableView() -> UIScrollView?
 }
 
-public protocol ParallaxViewDelegate: class {
+public protocol ParallaxViewDelegate: AnyObject {
   func parallaxViewDidScrollBy(percentage: CGFloat, oldOffset: CGPoint, newOffset: CGPoint)
 }