From b200f56497b16718191dfda2c4240eb87c6e43df Mon Sep 17 00:00:00 2001 From: Yee Cheng Chin Date: Fri, 20 Oct 2023 19:32:10 -0700 Subject: [PATCH] Fix non-native full screen show menu with notch, and changing resolution Currently MacVim's "show menu bar" option for non-native full screen does not work properly on newer MacBook's with a notch, as macOS's API for menu bar height for some reason does not return the full height of the menu bar (it only returns the height of the "normal" menu bar). We need to manually use visibleArea instead but that API also has an issue where it's for some reason 1 pixel less, meaning you see a black bar at the top, so we have to hack it to add 1 pixel. This is really not ideal and hopefully Apple fixes it with a new API. Also fix it so changing resolution when MacVim is in non-native full screen will work properly. When working on this, identified an issue that the dock detection logic doesn't really work, as we don't really know if the current screen has the dock visible or not due to lack of API to query it. Just left a comment for future reference. --- runtime/doc/gui_mac.txt | 13 +++++--- src/MacVim/MMFullScreenWindow.m | 55 +++++++++++++++++++++++++-------- src/MacVim/MMWindowController.m | 12 ++++++- 3 files changed, 61 insertions(+), 19 deletions(-) diff --git a/runtime/doc/gui_mac.txt b/runtime/doc/gui_mac.txt index d7a9d2fe72..ec1efe3256 100644 --- a/runtime/doc/gui_mac.txt +++ b/runtime/doc/gui_mac.txt @@ -370,11 +370,14 @@ MacVim can be used in full screen mode, see 'fullscreen'. There are two types of full screen modes. By default, MacVim uses macOS' native full screen functionality, which creates a separate space in Mission -Control. MacVim also provides a non-native full screen mode, which can be set -by disabling native full screen in the settings panel, or by setting -|MMNativeFullScreen| to `NO` manually. If you have a MacBook with a "notch" -at the top of the screen, you can set |MMNonNativeFullScreenShowMenu| to `NO` -and |MMNonNativeFullScreenSafeAreaBehavior| to 1 to utilitize the whole screen +Control. + +MacVim also provides a non-native full screen mode, which can be set by +disabling native full screen in the settings panel (see |MMNativeFullScreen|). +Use 'fuoptions' to configure the background color and whether to maximize the +rows/columns. If you have a MacBook with a camera housing ("notch") at the +top of the screen, you can set |MMNonNativeFullScreenShowMenu| to `NO` and +|MMNonNativeFullScreenSafeAreaBehavior| to 1 to utilitize the whole screen (this will cause some of the content to be obscured by the notch). ============================================================================== diff --git a/src/MacVim/MMFullScreenWindow.m b/src/MacVim/MMFullScreenWindow.m index 274b4cc5d2..03d55ed965 100644 --- a/src/MacVim/MMFullScreenWindow.m +++ b/src/MacVim/MMFullScreenWindow.m @@ -376,29 +376,48 @@ - (void)applicationDidChangeScreenParameters:(NSNotification *)notification [self setFrame:[screen frame] display:NO]; } -/// Get the view offset to allow us space to show the menu bar, or account for "safe area" (a.k.a. notch) in certain MacBook Pro's. +/// Get the view offset to allow us space to show the menu bar, or account for "safe area" (a.k.a. +/// notch) in certain MacBook Pro's. - (NSEdgeInsets) viewOffset { NSEdgeInsets offset = NSEdgeInsetsMake(0, 0, 0, 0); + NSUserDefaults *ud = [NSUserDefaults standardUserDefaults]; + const BOOL showMenu = [ud boolForKey:MMNonNativeFullScreenShowMenuKey]; + #if (MAC_OS_X_VERSION_MAX_ALLOWED >= MAC_OS_VERSION_12_0) // Account for newer MacBook Pro's which have a notch, which can be queried using the safe area API. if (@available(macos 12.0, *)) { - const int safeAreaBehavior = [[NSUserDefaults standardUserDefaults] - integerForKey:MMNonNativeFullScreenSafeAreaBehaviorKey]; - - // The safe area utilization is configuration. Right now, we only have two choices. - // In the future there may be more, e.g. showing tabs in the safe area. - if (safeAreaBehavior == 0) { - offset = [[self screen] safeAreaInsets]; + const int safeAreaBehavior = [ud integerForKey:MMNonNativeFullScreenSafeAreaBehaviorKey]; + + // The safe area utilization is configuration. Right now, we only have two choices: + // - 0: avoid the safe area (default) + // - 1: draw into the safe area, which would cause some contents to be obscured. + // In the future there may be more. E.g. we can draw tabs in the safe area. + // If menu is shown, we ignore this because this doesn't make sense. + if (safeAreaBehavior == 0 || showMenu) { + offset = [self screen].safeAreaInsets; } } #endif - if ([[NSUserDefaults standardUserDefaults] - boolForKey:MMNonNativeFullScreenShowMenuKey]) { - const CGFloat menuBarHeight = [[[NSApplication sharedApplication] mainMenu] menuBarHeight]; - if (menuBarHeight > offset.top) { - offset.top = menuBarHeight; + if (showMenu) { + // Offset by menu height + if (offset.top == 0) { + const CGFloat menuBarHeight = [[[NSApplication sharedApplication] mainMenu] menuBarHeight]; + if (menuBarHeight > offset.top) { + offset.top = menuBarHeight; + } + } else { + // Unfortunately, if there is a notch (safe area != 0), menuBarHeight does *not* return + // the menu height shown in the main screen, so we need to calculate it otherwise. + // visibleArea is supposed to give us this information but it's oddly off by one, leading + // to a one-pixel black line, so we need to manually increment it by one. Yes, it sucks. + NSRect visibleFrame = [self screen].visibleFrame; + visibleFrame.size.height += 1; + const CGFloat menuBarHeight = [self screen].frame.size.height - NSMaxY(visibleFrame); + if (menuBarHeight > offset.top) { + offset.top = menuBarHeight; + } } } @@ -482,6 +501,16 @@ - (BOOL)isOnPrimaryScreen return selfScreenNum == primaryScreenNum; } +/// Returns true when this screen has a dock and menu shown. +/// +/// @note +/// This does not reliably detect whether the dock is on the current screen or +/// not as there is no API to reliably detect this. We are mostly guessing here +/// but if the user sets the dock to display on left/right on a horizontal +/// layout, it may be on the other screen. +/// Also, technically when not using separate spaces, it's possible for the +/// menu to be on one screen and dock on the other. +/// This should be revisited in the future. - (BOOL)screenHasDockAndMenu { return NSScreen.screensHaveSeparateSpaces || [self isOnPrimaryScreen]; diff --git a/src/MacVim/MMWindowController.m b/src/MacVim/MMWindowController.m index 3d5b41df05..9f7789fb74 100644 --- a/src/MacVim/MMWindowController.m +++ b/src/MacVim/MMWindowController.m @@ -1278,7 +1278,17 @@ - (void)windowDidResize:(id)sender // Calling setFrameSizeKeepGUISize: instead of setFrameSize: prevents a // degenerate case where frameSizeMayHaveChanged: ends up resizing the window // *again* causing windowDidResize: to be called. - [vimView setFrameSizeKeepGUISize:[self contentSize]]; + if (fullScreenWindow == nil) { + [vimView setFrameSizeKeepGUISize:[self contentSize]]; + } else { + // Non-native full screen mode is more complicated and needs to + // re-layout the Vim view to properly account for the menu bar / notch, + // and misc fuopt configuration. + // This code is similar to what's done in processInputQueueDidFinish. + NSRect desiredFrame = [fullScreenWindow getDesiredFrame]; + [vimView setFrameOrigin:desiredFrame.origin]; + [vimView setFrameSizeKeepGUISize:desiredFrame.size]; + } } - (void)windowDidChangeBackingProperties:(NSNotification *)notification