From 1898f9162073910c05958295c612deec6121a892 Mon Sep 17 00:00:00 2001
From: Shelley Vohr <codebytere@github.com>
Date: Fri, 8 Feb 2019 18:07:08 -0800
Subject: [PATCH] fix: enable property having no effect on submenus (#16835)

---
 atom/browser/ui/cocoa/atom_menu_controller.mm | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/atom/browser/ui/cocoa/atom_menu_controller.mm b/atom/browser/ui/cocoa/atom_menu_controller.mm
index 695807135ebd..e28504a26a60 100644
--- a/atom/browser/ui/cocoa/atom_menu_controller.mm
+++ b/atom/browser/ui/cocoa/atom_menu_controller.mm
@@ -246,6 +246,11 @@ static base::scoped_nsobject<NSMenu> recentDocumentsMenuSwap_;
     [NSApp setServicesMenu:submenu];
   } else if (type == atom::AtomMenuModel::TYPE_SUBMENU &&
              model->IsVisibleAt(index)) {
+    // We need to specifically check that the submenu top-level item has been
+    // enabled as it's not validated by validateUserInterfaceItem
+    if (!model->IsEnabledAt(index))
+      [item setEnabled:NO];
+
     // Recursively build a submenu from the sub-model at this index.
     [item setTarget:nil];
     [item setAction:nil];