-
Notifications
You must be signed in to change notification settings - Fork 74
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
chore: Add various icons for Metals provided extensions and web views #1121
Conversation
src/doctor.ts
Outdated
@@ -39,6 +48,10 @@ export class DoctorProvider implements Disposable { | |||
ViewColumn.Active, | |||
{ enableCommandUris: true } | |||
); | |||
if (this.extensionContext) | |||
this.doctor.iconPath = Uri.file( | |||
path.join(this.extensionContext.extensionPath, "icons", "doctor.svg") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was thinking of having different doctors depending on the theme, but that seemed to be too much on the nose 😅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice idea!
] | ||
], | ||
"icon": { | ||
"dark": "icons/book-dark.svg", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it a default one for vscode?
I don't see it in pr
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It was actually already added previously, I reused an existing icon.
I realised while investigating how to add extension support for a single file that we can actually provide much better range of icons.
Java/Scala icons are the default vs code ones. The doctor and the stack trace icons are downloaded, but according to the website they are free to use for commercial/personal use.