From 27b4335084eec86e7d73851814d188a51ba59d04 Mon Sep 17 00:00:00 2001 From: "trop[bot]" Date: Wed, 20 Mar 2019 20:41:18 -0700 Subject: [PATCH] fix: take foreground visibility into account for isVisible on macOS (#17491) --- atom/browser/native_window_mac.mm | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/atom/browser/native_window_mac.mm b/atom/browser/native_window_mac.mm index 81f039dbcb2ba..5fe1e317fec6f 100644 --- a/atom/browser/native_window_mac.mm +++ b/atom/browser/native_window_mac.mm @@ -576,7 +576,12 @@ void ViewDidMoveToSuperview(NSView* self, SEL _cmd) { } bool NativeWindowMac::IsVisible() { - return [window_ isVisible]; + bool occluded = [window_ occlusionState] == NSWindowOcclusionStateVisible; + + // For a window to be visible, it must be visible to the user in the + // foreground of the app, which means that it should not be minimized or + // occluded + return [window_ isVisible] && !occluded && !IsMinimized(); } bool NativeWindowMac::IsEnabled() {