Skip to content

Commit c8be3bb

Browse files
committed
fine tune verified publisher changes
1 parent 1466122 commit c8be3bb

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

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

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -375,7 +375,7 @@ export class ExtensionEditor extends EditorPane {
375375
template.publisher.classList.toggle('clickable', !!extension.url);
376376
template.publisherDisplayName.textContent = extension.publisherDisplayName;
377377
template.verifiedPublisherIcon.style.display = extension.publisherDomain?.verified ? 'inherit' : 'none';
378-
template.publisher.title = extension.publisherDomain?.link ?? '';
378+
template.publisher.title = extension.publisherDomain?.link ? localize('publisher verified tooltip', "This publisher has verified ownership of {0}", URI.parse(extension.publisherDomain.link).authority) : '';
379379

380380
template.installCount.parentElement?.classList.toggle('hide', !extension.url);
381381

@@ -846,7 +846,8 @@ export class ExtensionEditor extends EditorPane {
846846
resources.push([localize('license', "License"), URI.parse(extension.licenseUrl)]);
847847
}
848848
if (extension.publisherDomain?.verified) {
849-
resources.push([extension.publisherDisplayName, URI.parse(extension.publisherDomain.link)]);
849+
const publisherDomainUri = URI.parse(extension.publisherDomain.link);
850+
resources.push([publisherDomainUri.authority, publisherDomainUri]);
850851
}
851852
if (resources.length) {
852853
const resourcesContainer = append(container, $('.resources-container'));

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

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -425,7 +425,8 @@ export class ExtensionHoverWidget extends ExtensionWidget {
425425

426426
if (this.extension.publisherDomain?.verified) {
427427
const bgColor = this.themeService.getColorTheme().getColor(extensionVerifiedPublisherIconColor);
428-
markdown.appendMarkdown(`<span style="color:${bgColor ? Color.Format.CSS.formatHex(bgColor) : '#ffffff'};">$(${verifiedPublisherIcon.id})</span>&nbsp;[${this.extension.publisherDomain.link.length > 50 ? `${this.extension.publisherDomain.link.substring(0, 50)}...` : this.extension.publisherDomain.link}](${this.extension.publisherDomain.link})`);
428+
const publisherVerifiedTooltip = localize('publisher verified tooltip', "This publisher has verified ownership of {0}", `[${URI.parse(this.extension.publisherDomain.link).authority}](${this.extension.publisherDomain.link})`);
429+
markdown.appendMarkdown(`<span style="color:${bgColor ? Color.Format.CSS.formatHex(bgColor) : '#ffffff'};">$(${verifiedPublisherIcon.id})</span>&nbsp;${publisherVerifiedTooltip}`);
429430
markdown.appendText(`\n`);
430431
}
431432

0 commit comments

Comments
 (0)