From 98e2a8bd3ca27a56150f1ea006a7a99e2c3fe23e Mon Sep 17 00:00:00 2001 From: Jordan Baird <jordanbaird.dev@gmail.com> Date: Sun, 14 Jul 2024 15:45:26 -0600 Subject: [PATCH] More descriptive comment --- Ice/MenuBar/MenuBarManager.swift | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/Ice/MenuBar/MenuBarManager.swift b/Ice/MenuBar/MenuBarManager.swift index ba23046d..061df71a 100644 --- a/Ice/MenuBar/MenuBarManager.swift +++ b/Ice/MenuBar/MenuBarManager.swift @@ -329,11 +329,10 @@ final class MenuBarManager: ObservableObject { return nil } - // workaround to prevent the clipping of menu items on secondary screens when the - // main screen has a notch, and the application menu extends past it... - // - // this isn't ideal, and arguably introduces a visual bug, but its a "better" bug - // than what it aims to fix + // the Accessibility API returns the menu bar for the active screen, regardless of + // the display origin used; this workaround prevents an incorrect frame from being + // returned for inactive displays in multi-display setups where one display has a + // notch if let mainScreen = NSScreen.main, let thisScreen = NSScreen.screens.first(where: { $0.displayID == displayID }),