fix: persist maximizable state when toggling fullscreen (#23019)

Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
electron · Apr 10, 2020 · 9984dd9 · 9984dd9
1 parent 7efa4cc
commit 9984dd9
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/shell/browser/ui/cocoa/atom_ns_window.mm b/shell/browser/ui/cocoa/atom_ns_window.mm
@@ -185,11 +185,17 @@ - (void)toggleFullScreenMode:(id)sender {
 
   // If we're in simple fullscreen mode and trying to exit it
   // we need to ensure we exit it properly to prevent a crash
-  // with NSWindowStyleMaskTitled mode
-  if (is_simple_fs || always_simple_fs)
+  // with NSWindowStyleMaskTitled mode.
+  if (is_simple_fs || always_simple_fs) {
     shell_->SetSimpleFullScreen(!is_simple_fs);
-  else
+  } else {
+    bool maximizable = shell_->IsMaximizable();
     [super toggleFullScreen:sender];
+
+    // Exiting fullscreen causes Cocoa to redraw the NSWindow, which resets
+    // the enabled state for NSWindowZoomButton. We need to persist it.
+    shell_->SetMaximizable(maximizable);
+  }
 }
 
 - (void)performMiniaturize:(id)sender {