Browse Source

Merge pull request #7962 from liusy182/disable-accelerator

disable accelerator when menu item is disabled
Kevin Sawicki 8 years ago
parent
commit
301b8ec39a
1 changed files with 5 additions and 4 deletions
  1. 5 4
      atom/browser/ui/accelerator_util.cc

+ 5 - 4
atom/browser/ui/accelerator_util.cc

@@ -90,11 +90,12 @@ bool TriggerAcceleratorTableCommand(AcceleratorTable* table,
                                     const ui::Accelerator& accelerator) {
   if (ContainsKey(*table, accelerator)) {
     const accelerator_util::MenuItem& item = (*table)[accelerator];
-    item.model->ActivatedAt(item.position);
-    return true;
-  } else {
-    return false;
+    if (item.model->IsEnabledAt(item.position)) {
+      item.model->ActivatedAt(item.position);
+      return true;
+    }
   }
+  return false;
 }
 
 }  // namespace accelerator_util