Skip to content

Commit a1b0e0e

Browse files
authored
Deprecated settings show up with undefined label (microsoft#162145)
Deprecated settings show up with undefined label
1 parent baf4f09 commit a1b0e0e

File tree

1 file changed

+9
-2
lines changed

1 file changed

+9
-2
lines changed

src/vs/workbench/contrib/extensions/browser/extensionEditor.ts

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1172,7 +1172,14 @@ export class ExtensionEditor extends EditorPane {
11721172
} else if (configuration) {
11731173
properties = configuration.properties;
11741174
}
1175-
const contrib = properties ? Object.keys(properties) : [];
1175+
1176+
let contrib = properties ? Object.keys(properties) : [];
1177+
1178+
// filter deprecated settings
1179+
contrib = contrib.filter(key => {
1180+
const config = properties[key];
1181+
return !config.deprecationMessage && !config.markdownDeprecationMessage;
1182+
});
11761183

11771184
if (!contrib.length) {
11781185
return false;
@@ -1187,7 +1194,7 @@ export class ExtensionEditor extends EditorPane {
11871194
$('th', undefined, localize('default', "Default"))
11881195
),
11891196
...contrib.map(key => {
1190-
let description: (Node | string) = properties[key].description;
1197+
let description: (Node | string) = properties[key].description || '';
11911198
if (properties[key].markdownDescription) {
11921199
const { element, dispose } = renderMarkdown({ value: properties[key].markdownDescription }, { actionHandler: { callback: (content) => this.openerService.open(content).catch(onUnexpectedError), disposables: this.contentDisposables } });
11931200
description = element;

0 commit comments

Comments
 (0)