Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Contributed icon in JS debug terminal stopped working #182303

Closed
Tyriar opened this issue May 12, 2023 · 2 comments · Fixed by #182615
Closed

Contributed icon in JS debug terminal stopped working #182303

Tyriar opened this issue May 12, 2023 · 2 comments · Fixed by #182615
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders terminal-profiles verified Verification succeeded
Milestone

Comments

@Tyriar
Copy link
Member

Tyriar commented May 12, 2023

See microsoft/vscode-js-debug#975 (comment)

I can reproduce:

image

@Tyriar Tyriar added bug Issue identified by VS Code Team member as probable bug terminal-profiles labels May 12, 2023
@Tyriar Tyriar added this to the Backlog milestone May 12, 2023
@tisilent
Copy link
Contributor

I want to try.

@Tyriar
Copy link
Member Author

Tyriar commented May 15, 2023

@tisilent sure! This is where the profile info is pulled in:

export class TerminalContributionService implements ITerminalContributionService {
declare _serviceBrand: undefined;
private _terminalProfiles: ReadonlyArray<IExtensionTerminalProfile> = [];
get terminalProfiles() { return this._terminalProfiles; }
constructor() {
terminalsExtPoint.setHandler(contributions => {
this._terminalProfiles = contributions.map(c => {
return c.value?.profiles?.filter(p => hasValidTerminalIcon(p)).map(e => {
return { ...e, extensionIdentifier: c.description.identifier.value };
}) || [];
}).flat();
});
}
}
function hasValidTerminalIcon(profile: ITerminalProfileContribution): boolean {
return !profile.icon ||
(
typeof profile.icon === 'string' ||
URI.isUri(profile.icon) ||
(
'light' in profile.icon && 'dark' in profile.icon &&
URI.isUri(profile.icon.light) && URI.isUri(profile.icon.dark)
)
);
}

tisilent added a commit to tisilent/vscode that referenced this issue May 16, 2023
@Tyriar Tyriar modified the milestones: Backlog, May 2023 May 19, 2023
Tyriar added a commit that referenced this issue May 19, 2023
Fix contributed profile icons
@VSCodeTriageBot VSCodeTriageBot added the unreleased Patch has not yet been released in VS Code Insiders label May 19, 2023
Tyriar added a commit that referenced this issue May 22, 2023
@VSCodeTriageBot VSCodeTriageBot added insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels May 25, 2023
@Tyriar Tyriar added the verified Verification succeeded label May 31, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Jul 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders terminal-profiles verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants