Revert "Update menu bar item getters"

This reverts commit 9c01743.
jordanbaird · Jul 22, 2024 · 8743c62 · 8743c62
1 parent f36177d
commit 8743c62
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 26 deletions.
diff --git a/Ice/Events/EventManager.swift b/Ice/Events/EventManager.swift
@@ -476,7 +476,7 @@ extension EventManager {
         else {
             return false
         }
-        let menuBarItems = MenuBarItem.getMenuBarItems(on: screen.displayID, using: .privateAPI, onScreenOnly: true)
+        let menuBarItems = MenuBarItem.getMenuBarItemsPrivateAPI(for: screen.displayID, onScreenOnly: true)
         return menuBarItems.contains { $0.frame.contains(mouseLocation) }
     }
 

diff --git a/Ice/MenuBar/MenuBarItem.swift b/Ice/MenuBar/MenuBarItem.swift
@@ -163,13 +163,8 @@ struct MenuBarItem {
 
 // MARK: MenuBarItem Getters
 extension MenuBarItem {
-    /// The method to use to get the menu bar items.
-    enum GetterMethod {
-        case coreGraphics
-        case privateAPI
-    }
-
-    private static func getMenuBarItemsCoreGraphics(on display: CGDirectDisplayID, onScreenOnly: Bool) -> [MenuBarItem] {
+    /// Returns an array of menu bar items in the menu bar for the given display.
+    static func getMenuBarItemsCoreGraphics(for display: CGDirectDisplayID, onScreenOnly: Bool) -> [MenuBarItem] {
         let windows = if onScreenOnly {
             WindowInfo.getOnScreenWindows(excludeDesktopWindows: true)
         } else {
@@ -193,7 +188,12 @@ extension MenuBarItem {
             .sortedByOrderInMenuBar()
     }
 
-    private static func getMenuBarItemsPrivateAPI(on display: CGDirectDisplayID, onScreenOnly: Bool) -> [MenuBarItem] {
+    /// Returns an array of menu bar items using private APIs to retrieve the
+    /// windows.
+    ///
+    /// - Parameter onScreenOnly: A Boolean value that indicates whether only
+    ///   the items that are on screen should be returned.
+    static func getMenuBarItemsPrivateAPI(for display: CGDirectDisplayID, onScreenOnly: Bool) -> [MenuBarItem] {
         var option: Bridging.WindowListOption = [.menuBarItems]
         if onScreenOnly {
             option.insert(.onScreen)
@@ -212,21 +212,7 @@ extension MenuBarItem {
             .sortedByOrderInMenuBar()
     }
 
-    /// Returns an array of menu bar items in the menu bar on the given display.
-    ///
-    /// - Parameters:
-    ///   - display: The display to retrieve the menu bar items on.
-    ///   - method: The method to use to get the items.
-    ///   - onScreenOnly: A Boolean value that indicates whether only the items
-    ///     that are on screen should be returned.
-    static func getMenuBarItems(on display: CGDirectDisplayID, using method: GetterMethod, onScreenOnly: Bool) -> [MenuBarItem] {
-        switch method {
-        case .coreGraphics: getMenuBarItemsCoreGraphics(on: display, onScreenOnly: onScreenOnly)
-        case .privateAPI: getMenuBarItemsPrivateAPI(on: display, onScreenOnly: onScreenOnly)
-        }
-    }
-
-    /// Returns an array of menu bar items using a private API to retrieve
+    /// Returns an array of menu bar items using private APIs to retrieve
     /// the windows.
     ///
     /// - Parameters:

diff --git a/Ice/MenuBar/MenuBarManager.swift b/Ice/MenuBar/MenuBarManager.swift
@@ -211,7 +211,7 @@ final class MenuBarManager: ObservableObject {
                         return
                     }
 
-                    let items = MenuBarItem.getMenuBarItems(on: displayID, using: .coreGraphics, onScreenOnly: true)
+                    let items = MenuBarItem.getMenuBarItemsCoreGraphics(for: displayID, onScreenOnly: true)
 
                     // get the leftmost item on the screen; the application menu should
                     // be hidden if the item's minX is close to the maxX of the menu

diff --git a/Ice/MenuBarAppearance/MenuBarOverlayPanel.swift b/Ice/MenuBarAppearance/MenuBarOverlayPanel.swift
@@ -541,7 +541,7 @@ private class MenuBarOverlayPanelContentView: NSView {
             return CGRect(x: rect.minX, y: rect.minY, width: maxX, height: rect.height)
         }()
         let trailingPathBounds: CGRect = {
-            let items = MenuBarItem.getMenuBarItems(on: screen.displayID, using: .privateAPI, onScreenOnly: true)
+            let items = MenuBarItem.getMenuBarItemsPrivateAPI(for: screen.displayID, onScreenOnly: true)
             guard !items.isEmpty else {
                 return .zero
             }