Skip to content

Commit 49f86e0

Browse files
committed
Ensure non-extension profiles get default icon
Follow up for microsoft#182303, see microsoft#182615
1 parent 558e126 commit 49f86e0

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

src/vs/workbench/contrib/terminal/browser/terminalInstance.ts

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -486,9 +486,14 @@ export class TerminalInstance extends Disposable implements ITerminalInstance {
486486
const defaultProfile = (await this._terminalProfileResolverService.getDefaultProfile({ remoteAuthority: this.remoteAuthority, os }));
487487
this.shellLaunchConfig.executable = defaultProfile.path;
488488
this.shellLaunchConfig.args = defaultProfile.args;
489-
// Only use default icon and color if they are undefined in the SLC
490-
this.shellLaunchConfig.icon ??= defaultProfile.icon;
491-
this.shellLaunchConfig.color ??= defaultProfile.color;
489+
if (this.shellLaunchConfig.isExtensionOwnedTerminal) {
490+
// Only use default icon and color if they are undefined in the SLC
491+
this.shellLaunchConfig.icon ??= defaultProfile.icon;
492+
this.shellLaunchConfig.color ??= defaultProfile.color;
493+
} else {
494+
this.shellLaunchConfig.icon = defaultProfile.icon;
495+
this.shellLaunchConfig.color = defaultProfile.color;
496+
}
492497
}
493498

494499
await this._createProcess();

0 commit comments

Comments
 (0)