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

[FEATURE] - Replace 'hardcoded' icons with product icons #719

Open
DeltaRazero opened this issue May 5, 2024 · 2 comments
Open

[FEATURE] - Replace 'hardcoded' icons with product icons #719

DeltaRazero opened this issue May 5, 2024 · 2 comments
Labels
enhancement side bar vscode api adoption Adopting new/updated VS Code APIs

Comments

@DeltaRazero
Copy link

DeltaRazero commented May 5, 2024

The icons currently used in the extension (activity bar icon and the gutter icon) are currently not themable by product icon themes. If these could be changed to use the product icon API then product icon themes can override them if desirable (i.e. to make them match the theme).

See aeschli's comment here microsoft/vscode#211811 (comment)

I can provide a pull request for providing a script for generating a product icon font, adding the contribution entries for the icons and change the activity bar icon to use the product icon API, however I cannot access the vscode-bookmarks-core repository to submit a pull-request so the gutter icon, so in that case you'll have to do that.

Awaiting your reply.

@alefragnani
Copy link
Owner

Hi @DeltaRazero ,

I thought about embracing Product Icon in the past, right after the API was released, but the problem (or limitation) is that it lacks support for workbench.colorCustomizations. So, the gutter icons wouldn't be as customizable as I want it to be. The Activity Bar icon, on the other hand, could become a product icon, for sure.

No need to worry about a PR. I have played with the sample available in vscode-extensions-sample repo in the past, and I have memories it was fairly easy to use it. It shouldn't have changed that much since then.

Thanks for your suggestion.

@alefragnani alefragnani added side bar vscode api adoption Adopting new/updated VS Code APIs labels May 7, 2024
@DeltaRazero
Copy link
Author

Thank you for reply.

Too bad to hear that the product icon API doesn't play well with the color customization aspect needed for the bookmarks extension, hopefully it's something that could be changed in the future.

However, even if only the activity bar icon can be a product icon, it's still a very welcoming change. I'm in the middle of making a file icon theme and product icon theme, and I'd like to support the most popular extensions or those I use personally so they fit right into the theme.

If no further PR or actions on my side are needed, then I'll leave the issue thread as is.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement side bar vscode api adoption Adopting new/updated VS Code APIs
Projects
None yet
Development

No branches or pull requests

2 participants