Skip to content

Commit

Permalink
warn when alt-command isn't supported, #8435
Browse files Browse the repository at this point in the history
  • Loading branch information
jrieken committed Jun 29, 2016
1 parent b4df3ad commit 428031b
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/vs/platform/actions/browser/menusExtensionPoint.ts
Original file line number Diff line number Diff line change
Expand Up @@ -260,7 +260,11 @@ ExtensionsRegistry.registerExtensionPoint<{ [loc: string]: IDeclaredMenuItem[] }
collector.warn(localize('missing.altCommand', "Menu item references an alt-command `{0}` which is not defined in the 'commands' section.", item.alt));
}
if (item.command === item.alt) {
collector.info(localize('dupe.command', "Menu item references the same command as default and alternative command"));
collector.info(localize('dupe.command', "Menu item references the same command as default and alt-command"));
}

if (item.alt && menu !== MenuId.EditorTitle) {
collector.info(localize('nosupport.altCommand', "Sorry, but currently only 'editor/title' supports alt-commands"));
}
}

Expand Down

0 comments on commit 428031b

Please sign in to comment.