Explicitly add refresh control as subview of scroll view, and use con…

…straints for its positioning.
hotwired · May 17, 2023 · a9c19d7 · a9c19d7
1 parent 3847dcb
commit a9c19d7
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/Source/Visitable/VisitableView.swift b/Source/Visitable/VisitableView.swift
@@ -45,6 +45,7 @@ open class VisitableView: UIView {
 
     open lazy var refreshControl: UIRefreshControl = {
         let refreshControl = UIRefreshControl()
+        refreshControl.translatesAutoresizingMaskIntoConstraints = false
         refreshControl.addTarget(self, action: #selector(refresh(_:)), for: .valueChanged)
         return refreshControl
     }()
@@ -67,13 +68,21 @@ open class VisitableView: UIView {
         guard let scrollView = webView?.scrollView, allowsPullToRefresh else { return }
 
         #if !targetEnvironment(macCatalyst)
-        scrollView.refreshControl = refreshControl
+        scrollView.addSubview(refreshControl)
+
+        let height = refreshControl.frame.height > 0 ? refreshControl.frame.height : 60
+
+        NSLayoutConstraint.activate([
+            refreshControl.centerXAnchor.constraint(equalTo: centerXAnchor),
+            refreshControl.topAnchor.constraint(equalTo: safeAreaLayoutGuide.topAnchor),
+            refreshControl.heightAnchor.constraint(equalToConstant: height)
+        ])
         #endif
     }
 
     private func removeRefreshControl() {
         refreshControl.endRefreshing()
-        webView?.scrollView.refreshControl = nil
+        refreshControl.removeFromSuperview()
     }
 
     @objc func refresh(_ sender: AnyObject) {