Skip to content

Commit

Permalink
no more proposed check for theme icons for menus, #84695
Browse files Browse the repository at this point in the history
  • Loading branch information
jrieken committed Dec 17, 2019
1 parent fd44b94 commit 9e0cffd
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions src/vs/workbench/api/common/menusExtensionPoint.ts
Original file line number Diff line number Diff line change
Expand Up @@ -351,11 +351,8 @@ commandsExtensionPoint.setHandler(extensions => {
let absoluteIcon: { dark: URI; light?: URI; } | ThemeIcon | undefined;
if (icon) {
if (typeof icon === 'string') {
if (extension.description.enableProposedApi) {
absoluteIcon = ThemeIcon.fromString(icon) || { dark: resources.joinPath(extension.description.extensionLocation, icon) };
} else {
absoluteIcon = { dark: resources.joinPath(extension.description.extensionLocation, icon) };
}
absoluteIcon = ThemeIcon.fromString(icon) || { dark: resources.joinPath(extension.description.extensionLocation, icon) };

} else {
absoluteIcon = {
dark: resources.joinPath(extension.description.extensionLocation, icon.dark),
Expand Down

0 comments on commit 9e0cffd

Please sign in to comment.