✨ Ability to adjust minimum screen size for padding

MrKai77 · Jul 9, 2024 · 0f96d0f · 0f96d0f
1 parent 3c7a9c8
commit 0f96d0f
Show file tree

Hide file tree

Showing 5 changed files with 25 additions and 10 deletions.
diff --git a/Loop/Extensions/Defaults+Extensions.swift b/Loop/Extensions/Defaults+Extensions.swift
@@ -101,3 +101,11 @@ extension Defaults.Keys {
     static let excludedApps = Key<[URL]>("excludedApps", default: [], iCloud: true)
     static let sizeIncrement = Key<CGFloat>("sizeIncrement", default: 20, iCloud: true)
 }
+
+// MARK: - Extra Advanced
+
+extension Defaults.Keys {
+    /// Adjust with `defaults write com.MrKai77.Loop paddingMinimumScreenSize -float x`
+    /// Reset with `defaults delete com.MrKai77.Loop paddingMinimumScreenSize`
+    static let paddingMinimumScreenSize = Key<CGFloat>("paddingMinimumScreenSize", default: 0, iCloud: true)
+}
diff --git a/Loop/Luminare/Settings/Behavior/BehaviorConfiguration.swift b/Loop/Luminare/Settings/Behavior/BehaviorConfiguration.swift
@@ -60,9 +60,7 @@ class BehaviorConfigurationModel: ObservableObject {
     }
 
     @Published var focusWindowOnResize = Defaults[.focusWindowOnResize] {
-        didSet {
-            Defaults[.focusWindowOnResize] = focusWindowOnResize
-        }
+        didSet { Defaults[.focusWindowOnResize] = focusWindowOnResize }
     }
 
     @Published var respectStageManager = Defaults[.respectStageManager] {

diff --git a/Loop/Preview Window/PreviewController.swift b/Loop/Preview Window/PreviewController.swift
@@ -91,7 +91,8 @@ class PreviewController {
 
         let targetWindowFrame = action.getFrame(
             window: window,
-            bounds: screen.safeScreenFrame
+            bounds: screen.safeScreenFrame,
+            screen: screen
         )
         .flipY(maxY: NSScreen.screens[0].frame.maxY)
 

diff --git a/Loop/Window Management/WindowAction.swift b/Loop/Window Management/WindowAction.swift
@@ -114,12 +114,13 @@ struct WindowAction: Codable, Identifiable, Hashable, Equatable, Defaults.Serial
         return result.normalized()
     }
 
-    func getFrame(window: Window?, bounds: CGRect, disablePadding: Bool = false) -> CGRect {
+    func getFrame(window: Window?, bounds: CGRect, disablePadding: Bool = false, screen: NSScreen? = nil) -> CGRect {
         guard direction != .cycle, direction != .noAction else {
             return NSRect(origin: bounds.center, size: .zero)
         }
         var bounds = bounds
-        if !disablePadding && Defaults[.enablePadding] {
+        if !disablePadding && Defaults[.enablePadding],
+           Defaults[.paddingMinimumScreenSize] == .zero || screen?.diagonalSize ?? .zero > Defaults[.paddingMinimumScreenSize] {
             bounds = getPaddedBounds(bounds)
         }
         var result = CGRect(origin: bounds.origin, size: .zero)
@@ -223,7 +224,7 @@ struct WindowAction: Codable, Identifiable, Hashable, Equatable, Defaults.Serial
 
         if !disablePadding {
             if direction != .undo, direction != .initialFrame {
-                result = cropThenApplyPadding(result, bounds)
+                result = cropThenApplyInnerPadding(result, bounds)
             }
 
             LoopManager.lastTargetFrame = result
@@ -398,13 +399,19 @@ struct WindowAction: Codable, Identifiable, Hashable, Equatable, Defaults.Serial
         return bounds
     }
 
-    private func cropThenApplyPadding(_ windowFrame: CGRect, _ bounds: CGRect) -> CGRect {
+    private func cropThenApplyInnerPadding(_ windowFrame: CGRect, _ bounds: CGRect, _ screen: NSScreen? = nil) -> CGRect {
         guard !direction.willMove else {
             return windowFrame
         }
 
         var croppedWindowFrame = windowFrame.intersection(bounds)
 
+        let paddingMinimumScreenSize = Defaults[.paddingMinimumScreenSize]
+        if paddingMinimumScreenSize != .zero,
+           screen?.diagonalSize ?? .zero < paddingMinimumScreenSize {
+            return windowFrame
+        }
+
         guard
             !willManipulateExistingWindowFrame,
             Defaults[.enablePadding]

diff --git a/Loop/Window Management/WindowEngine.swift b/Loop/Window Management/WindowEngine.swift
@@ -51,7 +51,7 @@ enum WindowEngine {
             return
         }
 
-        let targetFrame = action.getFrame(window: window, bounds: screen.safeScreenFrame)
+        let targetFrame = action.getFrame(window: window, bounds: screen.safeScreenFrame, screen: screen)
 
         if action.direction == .undo {
             WindowRecords.removeLastAction(for: window)
@@ -86,7 +86,8 @@ enum WindowEngine {
 
         let screenFrame = action.direction.willMove ? .zero : screen.safeScreenFrame
 
-        let bounds = if Defaults[.enablePadding] {
+        let bounds = if Defaults[.enablePadding],
+                        Defaults[.paddingMinimumScreenSize] == 0 || screen.diagonalSize > Defaults[.paddingMinimumScreenSize] {
             Defaults[.padding].apply(on: screenFrame)
         } else {
             screenFrame