Skip to content

Commit 92d8e1d

Browse files
author
Rachel Macfarlane
committed
Fix issue filing on theme extensions, fixes microsoft#46771
1 parent 7cf6969 commit 92d8e1d

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

src/vs/code/electron-browser/issue/issueReporterMain.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,7 @@ export class IssueReporter extends Disposable {
210210
});
211211

212212
const numberOfThemeExtesions = themes && themes.length;
213-
this.issueReporterModel.update({ numberOfThemeExtesions, enabledNonThemeExtesions: nonThemes });
213+
this.issueReporterModel.update({ numberOfThemeExtesions, enabledNonThemeExtesions: nonThemes, allExtensions: extensions });
214214
this.updateExtensionTable(nonThemes, numberOfThemeExtesions);
215215

216216
if (this.environmentService.disableExtensions || extensions.length === 0) {
@@ -786,7 +786,7 @@ export class IssueReporter extends Disposable {
786786

787787
this.addEventListener('extension-selector', 'change', (e: Event) => {
788788
const selectedExtensionId = (<HTMLInputElement>e.target).value;
789-
const extensions = this.issueReporterModel.getData().enabledNonThemeExtesions;
789+
const extensions = this.issueReporterModel.getData().allExtensions;
790790
const matches = extensions.filter(extension => extension.identifier.id === selectedExtensionId);
791791
if (matches.length) {
792792
this.issueReporterModel.update({ selectedExtension: matches[0] });

src/vs/code/electron-browser/issue/issueReporterModel.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ export interface IssueReporterData {
2626
includeSettingsSearchDetails?: boolean;
2727

2828
numberOfThemeExtesions?: number;
29+
allExtensions?: ILocalExtension[];
2930
enabledNonThemeExtesions?: ILocalExtension[];
3031
extensionsDisabled?: boolean;
3132
fileOnExtension?: boolean;

0 commit comments

Comments
 (0)