Fix the successful redeem as well

mullvad · Sep 21, 2023 · 55263e1 · 55263e1
1 parent c8ddb7a
commit 55263e1
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/ios/MullvadVPN/UI appearance/UIMetrics.swift b/ios/MullvadVPN/UI appearance/UIMetrics.swift
@@ -39,6 +39,7 @@ enum UIMetrics {
         static let cornerRadius = 8.0
         static let preferredContentSize = CGSize(width: 280, height: 260)
         static let contentLayoutMargins = NSDirectionalEdgeInsets(top: 16, leading: 16, bottom: 16, trailing: 16)
+        static let successfulRedeemMargins = NSDirectionalEdgeInsets(top: 16, leading: 8, bottom: 16, trailing: 8)
     }
 
     enum AccountDeletion {

diff --git a/ios/MullvadVPN/View controllers/RedeemVoucher/AddCreditSucceededViewController.swift b/ios/MullvadVPN/View controllers/RedeemVoucher/AddCreditSucceededViewController.swift
@@ -97,19 +97,18 @@ class AddCreditSucceededViewController: UIViewController, RootContainment {
 
     override func viewDidLoad() {
         super.viewDidLoad()
-        addSubviews()
-        addConstraints()
+        configureUI()
         addDismissButtonHandler()
     }
 
-    private func addSubviews() {
-        for subview in [statusImageView, titleLabel, messageLabel, dismissButton] {
-            view.addSubview(subview)
+    private func configureUI() {
+        let contentHolderView = UIView(frame: .zero)
+
+        view.addConstrainedSubviews([contentHolderView]) {
+            contentHolderView.pinEdgesToSuperview(.all(UIMetrics.SettingsRedeemVoucher.successfulRedeemMargins))
         }
-    }
 
-    private func addConstraints() {
-        view.addConstrainedSubviews([statusImageView, titleLabel, messageLabel, dismissButton]) {
+        contentHolderView.addConstrainedSubviews([statusImageView, titleLabel, messageLabel, dismissButton]) {
             statusImageView.pinEdgesToSuperviewMargins(PinnableEdges([.top(0)]))
             statusImageView.centerXAnchor.constraint(equalTo: view.centerXAnchor)