Keep the keyboard input-source icon visible

The hiding boundary sat right of the input-source indicator, so the
hamburger took the slot where the language icon had been. Move the
boundary left of it so the language icon stays, with the hamburger still
at the left edge.

💘 Generated with Crush

Assisted-by: Crush:deepseek-v4.1-flash

README.md

@@ -34,14 +34,12 @@ automatically.
 
 What it can and cannot hide:
 
-- It hides **other third-party apps' icons** (cmux, Kapture, WireGuard, input
+- It hides **other third-party apps' icons** (cmux, Kapture, moomoo, WireGuard).
-  source, Now Playing, …) so only your apps' icons leave the bar.
+- It **keeps Wi-Fi, Now Playing, the input-source indicator, Spotlight, Clock,
-- It **keeps Wi-Fi, Spotlight, Clock, Battery and Control Center**. The spacer's
+  Battery and Control Center** visible, with the **hamburger at the left edge of
-  boundary is tuned (preferred position 250) to fall just left of Wi-Fi, so the
+  that group**. The spacer's boundary (preferred position 330) falls just left
-  Apple items on the right stay visible. macOS reserves that far-right area: a
+  of the input-source indicator. macOS reserves the far-right area, so an item
-  third-party item can never be placed to the right of it, so those icons cannot
+  can never be placed to the right of the system icons.
-  be pushed away even in principle. (Bartender and Ice hit the same wall and
-  cover the area with their own overlay bar instead.)
 - Newly launched apps are placed to the left of the spacer, so they are hidden
   automatically.
 - Advanced: set `defaults write dev.tzgyn.simpletopbar debug.spacerPos -float <n>`

Sources/SimpleTopBar/App.swift

@@ -103,9 +103,13 @@ final class AppDelegate: NSObject, NSApplicationDelegate {
         let name = "SimpleTopBar"
         let positionKey = "NSStatusItem Preferred Position \(name)"
         // Only seed the position on first run so the user's own drag wins and
-        // persists through the autosave name.
+        // persists through the autosave name. Kept just below the hider's
-        if UserDefaults.standard.object(forKey: positionKey) == nil {
+        // preferred position so the hamburger lands at the left edge of the
-            UserDefaults.standard.set(100.0, forKey: positionKey)
+        // visible icons. `debug.hamburgerPos` overrides it.
+        if let override = UserDefaults.standard.object(forKey: "debug.hamburgerPos") as? Double {
+            UserDefaults.standard.set(override, forKey: positionKey)
+        } else if UserDefaults.standard.object(forKey: positionKey) == nil {
+            UserDefaults.standard.set(320.0, forKey: positionKey)
         }
 
         let item = NSStatusBar.system.statusItem(withLength: NSStatusItem.variableLength)

Sources/SimpleTopBar/MenuBarItems.swift

@@ -11,6 +11,8 @@ struct MenuBarItem: Identifiable, Equatable {
     let icon: NSImage
     /// True for Apple's own items (Control Center, Spotlight, …).
     let isSystem: Bool
+    /// Frame in Accessibility coordinates (origin top-left of the main screen).
+    let frame: CGRect?
     let element: AXUIElement
 
     static func == (lhs: MenuBarItem, rhs: MenuBarItem) -> Bool {
@@ -75,10 +77,11 @@ final class MenuBarItemManager: ObservableObject {
 
             var seenTitles = Set<String>()
             for (index, child) in children.enumerated() {
+                let position = pointAttribute(child, kAXPositionAttribute)
                 // Control Center reports every module it owns, including the
                 // ones not in the menu bar, which are parked off-screen around
                 // y=982. Keep only items actually sitting in the bar.
-                if let position = pointAttribute(child, kAXPositionAttribute), position.y > 60 {
+                if let position, position.y > 60 {
                     continue
                 }
 
@@ -90,6 +93,14 @@ final class MenuBarItemManager: ObservableObject {
                 // "Control Center"; show each distinct title once per app.
                 if !seenTitles.insert(title).inserted { continue }
 
+                let size = sizeAttribute(child, kAXSizeAttribute)
+                let frame: CGRect?
+                if let position, let size {
+                    frame = CGRect(origin: position, size: size)
+                } else {
+                    frame = nil
+                }
+
                 found.append(MenuBarItem(
                     id: "\(pid)-\(index)",
                     pid: pid,
@@ -97,6 +108,7 @@ final class MenuBarItemManager: ObservableObject {
                     title: title,
                     icon: Self.icon(forTitle: title, app: app),
                     isSystem: Self.isSystemBundle(app.bundleIdentifier),
+                    frame: frame,
                     element: child
                 ))
             }
@@ -138,12 +150,27 @@ final class MenuBarItemManager: ObservableObject {
     }
 
     private func performPress(_ item: MenuBarItem) -> Bool {
+        // Spotlight's status item accepts the accessibility press action but
+        // does nothing with it; it needs a genuine click.
+        if item.appName.lowercased().contains("spotlight"), let frame = item.frame {
+            postClick(at: CGPoint(x: frame.midX, y: frame.midY))
+            return true
+        }
         if AXUIElementPerformAction(item.element, kAXPressAction as CFString) == .success {
             return true
         }
         return AXUIElementPerformAction(item.element, kAXShowMenuAction as CFString) == .success
     }
 
+    /// Posts a real left click at a point in Accessibility/global coordinates.
+    private func postClick(at point: CGPoint) {
+        let source = CGEventSource(stateID: .hidSystemState)
+        let down = CGEvent(mouseEventSource: source, mouseType: .leftMouseDown, mouseCursorPosition: point, mouseButton: .left)
+        let up = CGEvent(mouseEventSource: source, mouseType: .leftMouseUp, mouseCursorPosition: point, mouseButton: .left)
+        down?.post(tap: .cghidEventTap)
+        up?.post(tap: .cghidEventTap)
+    }
+
     // MARK: - Classification
 
     /// Apple's own menu bar items (Control Center, Spotlight, the input-source
@@ -235,6 +262,16 @@ final class MenuBarItemManager: ObservableObject {
         guard AXValueGetValue(value as! AXValue, .cgPoint, &point) else { return nil }
         return point
     }
+
+    nonisolated static func sizeAttribute(_ element: AXUIElement, _ attribute: String) -> CGSize? {
+        var value: CFTypeRef?
+        guard AXUIElementCopyAttributeValue(element, attribute as CFString, &value) == .success,
+              let value,
+              CFGetTypeID(value) == AXValueGetTypeID() else { return nil }
+        var size = CGSize.zero
+        guard AXValueGetValue(value as! AXValue, .cgSize, &size) else { return nil }
+        return size
+    }
 }
 
 /// Best-effort menu bar item hiding. A zero-width spacer status item is created
@@ -276,10 +313,15 @@ final class MenuBarHider: ObservableObject {
         // The boundary between "hidden" and "kept" falls just left of Wi-Fi at
         // ~250, so the other apps hide while Apple's Wi-Fi/Clock/Battery/Control
         // Center/Spotlight stay visible. `debug.spacerPos` overrides the seed.
+        // The boundary sits just left of the input-source indicator, so every
+        // system item (input source, Now Playing, Wi-Fi, …) stays visible while
+        // the apps hide. It must stay *above* the hamburger's preferred position
+        // (see AppDelegate) for the hamburger to remain on the visible side.
+        // `debug.spacerPos` overrides the seed.
         if let override = UserDefaults.standard.object(forKey: "debug.spacerPos") as? Double {
             UserDefaults.standard.set(override, forKey: key)
         } else if UserDefaults.standard.object(forKey: key) == nil {
-            UserDefaults.standard.set(250.0, forKey: key)
+            UserDefaults.standard.set(330.0, forKey: key)
         }
         let item = NSStatusBar.system.statusItem(withLength: 0)
         item.autosaveName = name

Sources/SimpleTopBar/SelfTest.swift

@@ -103,9 +103,9 @@ func runSelfTest() {
     check(MenuBarItemManager.children(of: systemWide) == nil, "no children on a non-app element")
 
     let element = AXUIElementCreateSystemWide()
-    let first = MenuBarItem(id: "1-0", pid: 1, appName: "A", title: "T", icon: NSImage(), isSystem: false, element: element)
+    let first = MenuBarItem(id: "1-0", pid: 1, appName: "A", title: "T", icon: NSImage(), isSystem: false, frame: nil, element: element)
-    let same = MenuBarItem(id: "1-0", pid: 1, appName: "A", title: "T", icon: NSImage(), isSystem: false, element: element)
+    let same = MenuBarItem(id: "1-0", pid: 1, appName: "A", title: "T", icon: NSImage(), isSystem: false, frame: nil, element: element)
-    let different = MenuBarItem(id: "1-1", pid: 1, appName: "A", title: "T", icon: NSImage(), isSystem: false, element: element)
+    let different = MenuBarItem(id: "1-1", pid: 1, appName: "A", title: "T", icon: NSImage(), isSystem: false, frame: nil, element: element)
     check(first == same, "menu bar items compare by id and title")
     check(first != different, "different ids are not equal")
 

Sources/SimpleTopBar/TopBarView.swift

@@ -95,7 +95,7 @@ struct TopBarView: View {
                 .controlSize(.mini)
                 .padding(.horizontal, 12)
                 .padding(.top, 6)
-                .help("Best-effort: pushes other apps' icons off the menu bar so they only live here. Apple's Wi-Fi, Spotlight, Clock, Battery and Control Center stay visible; macOS won't let any third-party app move them.")
+                .help("Best-effort: pushes other apps' icons off the menu bar so they only live here. Apple's system items (Wi-Fi, Now Playing, Clock, Battery, Control Center, Spotlight) stay visible; macOS won't let any third-party app move them.")
             }
         }
     }

build-app.sh

@@ -2,7 +2,7 @@
 set -euo pipefail
 cd "$(dirname "$0")"
 
-VERSION="${1:-0.4.4}"
+VERSION="${1:-0.4.7}"
 
 swift build -c release