✨ Don't reset padding if all values are equivalent

MrKai77 · Jun 17, 2024 · 77cc326 · 77cc326
1 parent 1705c86
commit 77cc326
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 5 deletions.
diff --git a/Loop/Luminare/Settings/Behavior/Padding Configuration/PaddingConfigurationView.swift b/Loop/Luminare/Settings/Behavior/Padding Configuration/PaddingConfigurationView.swift
@@ -60,11 +60,19 @@ struct PaddingConfigurationView: View {
                         paddingModel.configureScreenPadding = newValue
 
                         if !paddingModel.configureScreenPadding {
-                            paddingModel.window = 0
-                            paddingModel.top = 0
-                            paddingModel.bottom = 0
-                            paddingModel.right = 0
-                            paddingModel.left = 0
+                            if paddingModel.allEqual {
+                                let window = paddingModel.window
+                                paddingModel.top = window
+                                paddingModel.bottom = window
+                                paddingModel.right = window
+                                paddingModel.left = window
+                            } else {
+                                paddingModel.window = 0
+                                paddingModel.top = 0
+                                paddingModel.bottom = 0
+                                paddingModel.right = 0
+                                paddingModel.left = 0
+                            }
                         }
                     }
                 }

diff --git a/Loop/Utilities/PaddingModel.swift b/Loop/Utilities/PaddingModel.swift
@@ -39,4 +39,8 @@ struct PaddingModel: Codable, Defaults.Serializable, Hashable {
     var totalHorizontalPadding: CGFloat {
         right + left
     }
+
+    var allEqual: Bool {
+        window == top && window == bottom && window == right && window == left
+    }
 }