From a277c154fd96579a2cf7c3b4757428a326ae9bb1 Mon Sep 17 00:00:00 2001
From: Kai Azim <hello@mrkai77.dev>
Date: Thu, 29 Aug 2024 23:44:08 -0600
Subject: [PATCH] =?UTF-8?q?=F0=9F=90=9E=20Fix=20few=20bugs=20in=20custom?=
 =?UTF-8?q?=20action=20configuration=20view?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../CustomActionConfigurationView.swift       | 42 ++++++++++++++-----
 1 file changed, 31 insertions(+), 11 deletions(-)

diff --git a/Loop/Luminare/Settings/Keybindings/CustomActionConfigurationView.swift b/Loop/Luminare/Settings/Keybindings/CustomActionConfigurationView.swift
index d67e947c..abce917b 100644
--- a/Loop/Luminare/Settings/Keybindings/CustomActionConfigurationView.swift
+++ b/Loop/Luminare/Settings/Keybindings/CustomActionConfigurationView.swift
@@ -81,7 +81,7 @@ struct CustomActionConfigurationView: View {
     }
 
     @ViewBuilder private func tabPicker() -> some View {
-        LuminarePicker(elements: Tab.allCases, selection: $currentTab, columns: 2, roundBottom: false) { tab in
+        LuminarePicker(elements: Tab.allCases, selection: $currentTab.animation(LuminareSettingsWindow.animation), columns: 2, roundBottom: false) { tab in
             HStack(spacing: 6) {
                 tab.image
                 Text(tab.rawValue)
@@ -91,26 +91,46 @@ struct CustomActionConfigurationView: View {
     }
 
     @ViewBuilder private func unitToggle() -> some View {
-        LuminareToggle("Use pixels", isOn: Binding(get: { action.unit == .pixels }, set: { action.unit = $0 ? .pixels : .percentage }))
+        LuminareToggle(
+            "Use pixels",
+            isOn: Binding(
+                get: {
+                    if action.unit == nil {
+                        action.unit = .percentage
+                    }
+
+                    return action.unit == .pixels
+                },
+                set: {
+                    action.unit = $0 ? .pixels : .percentage
+                }
+            )
+        )
     }
 
     @ViewBuilder private func actionButtons() -> some View {
         HStack(spacing: 8) {
-            Button("Preview") {
-                previewAction()
-            }
-            .disabled(action.sizeMode != .custom)
+            Button("Preview") {}
+                .onLongPressGesture( // Allows for a press-and-hold gesture to show the preview
+                    minimumDuration: 100.0,
+                    maximumDistance: .infinity,
+                    pressing: { pressing in
+                        if pressing {
+                            guard let screen = NSScreen.main else { return }
+                            previewController.open(screen: screen, startingAction: action)
+                        } else {
+                            previewController.close()
+                        }
+                    },
+                    perform: {}
+                )
+                .disabled(action.sizeMode != .custom)
 
             Button("Close") { isPresented = false }
         }
         .buttonStyle(LuminareCompactButtonStyle())
     }
 
-    private func previewAction() {
-        guard let screen = NSScreen.main else { return }
-        previewController.open(screen: screen, startingAction: action)
-    }
-
     @ViewBuilder private func positionConfiguration() -> some View {
         LuminareSection {
             LuminareToggle(