Release v0.14.0: popup shortcuts & pin window (#69)

* feat(Popup): improve translation popup shortcuts (#65)

* feat(Popup): enhance input focus handling and localization updates

* chore: bump version to 0.14.0

* feat: 增加翻译窗口右上角固定窗口置顶按钮 (#64)

* Add pinned popup dismiss hook

* Track pinned popup state

* Add popup pin button

* Refine popup pin placement

* Hide popup pin outside translation view

* fix: unpin error popup

* fix: handle missing screen recording permission

---------

Co-authored-by: yusixian <i@cosine.ren>

* refactor(Popup): centralize pin state and tighten async handoff

- Move `isPinned` into TranslationCoordinator as single source of truth;
  PopupView toggles, PopupPanelController reads. Remove duplicated @State
  and onPinnedChange callback plumbing.
- Auto-unpin via `globalError` didSet so error states never appear in a
  frozen pinned panel.
- Move pin button from ZStack overlay into the language bar HStack to
  stop it from occluding the source input's top-right corner.
- Defer first-responder reassignment in `consumePendingSourceInputFocus`
  via `Task { @mainactor }` to avoid re-entering AppKit during
  `viewDidMoveToWindow`.
- Cancelable `pulseTask` in ProviderResultCard with onDisappear cleanup
  to prevent overlapping copy-pulse animations on rapid copies.

---------

Co-authored-by: Qing Guo <qingeekk@qq.com>
Co-authored-by: JimssM <hydraliskm91@gmail.com>

Author: 3 people

Project.swift

@@ -49,7 +49,7 @@ let project = Project(
                     "CODE_SIGN_STYLE": "$(MOEPEEK_CODE_SIGN_STYLE)",
                     "CODE_SIGN_IDENTITY": "$(MOEPEEK_CODE_SIGN_IDENTITY)",
                     "DEVELOPMENT_TEAM": "$(MOEPEEK_DEVELOPMENT_TEAM)",
-                    "MARKETING_VERSION": "0.13.1",
+                    "MARKETING_VERSION": "0.14.0",
                     "CURRENT_PROJECT_VERSION": "1",
                 ],
                 configurations: [

Resources/Localizable.xcstrings

@@ -47,6 +47,7 @@
       }
     },
     "↵ Translate · ⇧↵ Newline" : {
+      "extractionState" : "stale",
       "localizations" : {
         "zh-Hans" : {
           "stringUnit" : {
@@ -56,6 +57,16 @@
         }
       }
     },
+    "↵ Translate · ⌘↵ Copy & Close · ⇧↵ Newline" : {
+      "localizations" : {
+        "zh-Hans" : {
+          "stringUnit" : {
+            "state" : "translated",
+            "value" : "↵ 翻译 · ⌘↵ 复制并关闭 · ⇧↵ 换行"
+          }
+        }
+      }
+    },
     "About" : {
       "extractionState" : "stale",
       "localizations" : {
@@ -1466,6 +1477,17 @@
         }
       }
     },
+    "Pin Popup" : {
+      "extractionState" : "stale",
+      "localizations" : {
+        "zh-Hans" : {
+          "stringUnit" : {
+            "state" : "translated",
+            "value" : "钉住窗口"
+          }
+        }
+      }
+    },
     "Please enter a valid Base URL (starting with http:// or https://) and API Key" : {
       "extractionState" : "stale",
       "localizations" : {
@@ -1778,6 +1800,17 @@
         }
       }
     },
+    "Screen recording permission not granted. Open Settings to enable it." : {
+      "extractionState" : "stale",
+      "localizations" : {
+        "zh-Hans" : {
+          "stringUnit" : {
+            "state" : "translated",
+            "value" : "屏幕录制权限未授权。请前往设置中开启。"
+          }
+        }
+      }
+    },
     "Screenshot OCR" : {
       "extractionState" : "stale",
       "localizations" : {
@@ -2285,6 +2318,17 @@
         }
       }
     },
+    "Unpin Popup" : {
+      "extractionState" : "stale",
+      "localizations" : {
+        "zh-Hans" : {
+          "stringUnit" : {
+            "state" : "translated",
+            "value" : "取消钉住窗口"
+          }
+        }
+      }
+    },
     "Unsupported" : {
       "extractionState" : "stale",
       "localizations" : {

Sources/Core/TranslationCoordinator.swift

@@ -26,18 +26,28 @@ final class TranslationCoordinator {
     private(set) var detectedLanguage: String?
     private(set) var targetLanguage: String = ""
     private(set) var providerStates: [String: ProviderState] = [:]
-    private(set) var globalError: String?
+    /// Setting a non-nil error also auto-unpins, so an error never appears in a frozen pinned panel.
+    private(set) var globalError: String? {
+        didSet {
+            if globalError != nil { isPinned = false }
+        }
+    }
     private(set) var detectionResult: DetectionResult?
     /// Snapshot of expanded provider slots for the current translation session.
     /// PopupView reads this instead of `registry.enabledSlots` to avoid recomputation during streaming.
     private(set) var activeSlots: [any TranslationProvider] = []
     /// Monotonically increasing counter; increments each time `translate()` is called.
     /// Used by PopupView to reset `expandedProviders` for subsequent translations.
     private(set) var translationGeneration: Int = 0
+    private(set) var copiedProviderID: String?
+    private(set) var copyFeedbackGeneration: Int = 0
+    /// Single source of truth for popup pin state. PopupView toggles, PopupPanelController reads.
+    var isPinned: Bool = false
 
     let registry: TranslationProviderRegistry
     private let permissionManager: PermissionManager
     private var activeTasks: [String: Task<Void, Never>] = [:]
+    private var copyFeedbackTask: Task<Void, Never>?
 
     init(permissionManager: PermissionManager, registry: TranslationProviderRegistry) {
         self.permissionManager = permissionManager
@@ -69,13 +79,21 @@ final class TranslationCoordinator {
 
     /// Triggered by OCR shortcut: screen capture → OCR → translate.
     func ocrAndTranslate() async {
+        guard permissionManager.isScreenRecordingGranted else {
+            phase = .active
+            sourceText = ""
+            globalError = String(localized: "Screen recording permission not granted. Open Settings to enable it.")
+            return
+        }
+
+        let previousPhase = phase
         phase = .grabbing
 
         do {
             let text = try await ScreenCaptureOCR.captureAndRecognize()
             translate(text)
-        } catch is OCRError {
-            phase = .idle
+        } catch OCRError.captureCancelled {
+            phase = previousPhase
         } catch {
             phase = .active
             sourceText = ""
@@ -98,6 +116,7 @@ final class TranslationCoordinator {
     /// Reset state and enter input mode (empty source input for manual typing).
     func prepareInputMode() {
         cancelAll()
+        clearCopyFeedback()
         globalError = nil
         sourceText = ""
         detectedLanguage = nil
@@ -120,6 +139,7 @@ final class TranslationCoordinator {
         }
 
         cancelAll()
+        clearCopyFeedback()
         globalError = nil
 
         sourceText = trimmed
@@ -182,8 +202,27 @@ final class TranslationCoordinator {
         activeTasks[provider.id] = task
     }
 
+    @discardableResult
+    func copyResult(atDisplayIndex index: Int) -> Bool {
+        guard activeSlots.indices.contains(index) else { return false }
+        return copyResult(forProviderID: activeSlots[index].id)
+    }
+
+    @discardableResult
+    func copyResult(forProviderID providerID: String) -> Bool {
+        guard let resultText = providerStates[providerID]?.copyableText,
+              !resultText.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty
+        else { return false }
+
+        NSPasteboard.general.clearContents()
+        guard NSPasteboard.general.setString(resultText, forType: .string) else { return false }
+        showCopyFeedback(forProviderID: providerID)
+        return true
+    }
+
     func dismiss() {
         cancelAll()
+        clearCopyFeedback()
         phase = .idle
         sourceText = ""
         detectedLanguage = nil
@@ -192,6 +231,7 @@ final class TranslationCoordinator {
         globalError = nil
         detectionResult = nil
         activeSlots = []
+        isPinned = false
     }
 
     // MARK: - Computed Helpers
@@ -260,6 +300,26 @@ final class TranslationCoordinator {
         activeTasks.removeAll()
     }
 
+    private func showCopyFeedback(forProviderID providerID: String) {
+        copyFeedbackTask?.cancel()
+        copiedProviderID = providerID
+        copyFeedbackGeneration += 1
+
+        copyFeedbackTask = Task { @MainActor [weak self] in
+            try? await Task.sleep(for: .milliseconds(900))
+            guard !Task.isCancelled else { return }
+            guard self?.copiedProviderID == providerID else { return }
+            self?.copiedProviderID = nil
+            self?.copyFeedbackTask = nil
+        }
+    }
+
+    private func clearCopyFeedback() {
+        copyFeedbackTask?.cancel()
+        copyFeedbackTask = nil
+        copiedProviderID = nil
+    }
+
     /// Build language hints (BCP 47 codes) based on user's target/source language preferences.
     /// Likely source languages are inferred from the target language for common translation pairs.
     private func buildLanguageHints() -> [String: Double]? {
@@ -329,3 +389,16 @@ final class TranslationCoordinator {
         return preferred
     }
 }
+
+private extension TranslationCoordinator.ProviderState {
+    var copyableText: String? {
+        switch self {
+        case let .streaming(partial):
+            partial
+        case let .completed(text):
+            text
+        case .waiting, .translating, .error:
+            nil
+        }
+    }
+}

Sources/UI/PopupDismissMonitor.swift

@@ -4,15 +4,21 @@ import AppKit
 @MainActor
 final class PopupDismissMonitor {
     private let panel: NSPanel
+    private let isPinned: @MainActor () -> Bool
     private let onDismiss: @MainActor () -> Void
 
     nonisolated(unsafe) private var globalClickMonitor: Any?
     nonisolated(unsafe) private var globalKeyMonitor: Any?
     nonisolated(unsafe) private var localClickMonitor: Any?
     nonisolated(unsafe) private var localKeyMonitor: Any?
 
-    init(panel: NSPanel, onDismiss: @escaping @MainActor () -> Void) {
+    init(
+        panel: NSPanel,
+        isPinned: @escaping @MainActor () -> Bool = { false },
+        onDismiss: @escaping @MainActor () -> Void
+    ) {
         self.panel = panel
+        self.isPinned = isPinned
         self.onDismiss = onDismiss
     }
 
@@ -22,7 +28,7 @@ final class PopupDismissMonitor {
             matching: [.leftMouseDown, .rightMouseDown]
         ) { [weak self] _ in
             Task { @MainActor in
-                self?.onDismiss()
+                self?.dismissForOutsideClick()
             }
         }
 
@@ -43,12 +49,12 @@ final class PopupDismissMonitor {
             let clickLocation = event.locationInWindow
             if event.window !== self.panel {
                 Task { @MainActor in
-                    self.onDismiss()
+                    self.dismissForOutsideClick()
                 }
             } else if let contentView = self.panel.contentView,
                       !contentView.frame.contains(clickLocation) {
                 Task { @MainActor in
-                    self.onDismiss()
+                    self.dismissForOutsideClick()
                 }
             }
             return event
@@ -65,6 +71,11 @@ final class PopupDismissMonitor {
         }
     }
 
+    private func dismissForOutsideClick() {
+        guard !isPinned() else { return }
+        onDismiss()
+    }
+
     func stop() {
         if let monitor = globalClickMonitor { NSEvent.removeMonitor(monitor) }
         if let monitor = globalKeyMonitor { NSEvent.removeMonitor(monitor) }