Fix landscape and SE bug

kickstarter · May 11, 2020 · 761dcb2 · 761dcb2
1 parent 8e98266
commit 761dcb2
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/Kickstarter-iOS/Views/Controllers/PledgeAmountViewController.swift b/Kickstarter-iOS/Views/Controllers/PledgeAmountViewController.swift
@@ -144,7 +144,7 @@ final class PledgeAmountViewController: UIViewController {
       }
   }
 
-  override func didMove(toParent parent: UIViewController?) {
+  override func didMove(toParent _: UIViewController?) {
     self.verticalSpacer.isHidden = true
   }
 

diff --git a/Kickstarter-iOS/Views/Controllers/PledgeViewController.swift b/Kickstarter-iOS/Views/Controllers/PledgeViewController.swift
@@ -170,6 +170,14 @@ final class PledgeViewController: UIViewController,
     self.sessionStartedObserver.doIfSome(NotificationCenter.default.removeObserver)
   }
 
+  override func viewDidAppear(_ animated: Bool) {
+    super.viewDidAppear(animated)
+
+    // Works around a layout bug when in landscape or on iPhone SE - last observed on iOS 13.4.
+    self.addChild(self.paymentMethodsViewController)
+    self.paymentMethodsViewController.didMove(toParent: self)
+  }
+
   // MARK: - Configuration
 
   private func configureChildViewControllers() {
@@ -189,8 +197,7 @@ final class PledgeViewController: UIViewController,
       self.pledgeAmountViewController,
       self.pledgeAmountSummaryViewController,
       self.shippingLocationViewController,
-      self.summaryViewController,
-      self.paymentMethodsViewController
+      self.summaryViewController
     ]
 
     let arrangedSubviews = [