mirrors/electron
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions 1

revert: allow NSMenuItems to be disabled (#46502)

Browse source 
Revert "fix: allow NSMenuItems to be disabled (#46307)"

This reverts commit ac616ef41d.
This commit is contained in:
Hailey 2025-04-06 18:42:05 -07:00 • committed by GitHub
parent 052da50aeb
commit ac18decc56
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 6 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

15
shell/browser/ui/cocoa/electron_menu_controller.mm
View file

@ -320,10 +320,6 @@ NSArray* ConvertSharingItemToNS(const SharingItem& item) {
NSMenuItem* item = [[NSMenuItem alloc] initWithTitle:label
action:@selector(itemSelected:)
keyEquivalent:@""];
if (model->IsEnabledAt(index))
[item setEnabled:YES];
else
[item setEnabled:NO];
// If the menu item has an icon, set it.
ui::ImageModel icon = model->GetIconAt(index);
@ -353,6 +349,11 @@ NSArray* ConvertSharingItemToNS(const SharingItem& item) {
[item setSubmenu:[self createShareMenuForItem:sharing_item]];
} else if (type == electron::ElectronMenuModel::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];
@ -492,10 +493,8 @@ NSArray* ConvertSharingItemToNS(const SharingItem& item) {
}
- (NSMenu*)menu {
if (menu_) {
[menu_ setAutoenablesItems:NO];
if (menu_)
return menu_;
}
if (model_ && model_->sharing_item()) {
NSMenu* menu = [self createShareMenuForItem:*model_->sharing_item()];
@ -505,8 +504,6 @@ NSArray* ConvertSharingItemToNS(const SharingItem& item) {
if (model_)
[self populateWithModel:model_.get()];
}
[menu_ setAutoenablesItems:NO];
[menu_ setDelegate:self];
return menu_;
}