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

Adding some file icons to the vs code extension #542

Closed
wants to merge 3 commits into from

Conversation

anwather
Copy link

@anwather anwather commented Sep 22, 2020

fixes #521
Added light and dark icons

@alex-frankel
Copy link
Collaborator

thank you @anwather for picking this up!

We have a few concerns about what icon assets we are allowed to use, so we'd prefer to work with our design team to get those assets generated. We also would prefer them to be SVGs if possible. Would you be alright if we provide you the assets, and then you can use those and update the PR?

@anwather
Copy link
Author

@alex-frankel Not a problem - thought that might be a concern but the groundwork is there 👍

@kilasuit
Copy link

incase you didn't know if you edit this comment and add

fixes #521 it will properly link that this PR should fix that issue & close it

this is how it looks in the issue (which is really helpful)
image

but more helpful it shows in the Issue list like this which helps people not raise multiple PR's against the same Issue
image

@anthony-c-martin anthony-c-martin changed the base branch from master to main October 16, 2020 16:39
@alex-frankel
Copy link
Collaborator

We mocked up a few potential options for this, curious to hear what folks think about these:

mechanical arm:
image

lowercase "b":
image

@kilasuit
Copy link

kilasuit commented Nov 9, 2020

+1 for Mechanical ARM

@anwather
Copy link
Author

I think I like the 'b' - but maybe raise an issue and pin it to get people to vote 👍

@anthony-c-martin
Copy link
Member

I dug a bit further into this, and it seems that the contribution point you're using in this PR ("iconThemes") is intended for a theming extension to provide a full set of icons to replace all file types in VSCode. So while this results in the bicep icon being assigned to .bicep files, it removes icons from all other files.

From my understanding, the correct way to get icons into VSCode is to contribute to the setui repo, which generates an iconfont that is periodically rolled into the main VSCode build. I've submitted a PR to add a .bicep icon here: jesseweed/seti-ui#586.

As such I'm going to close this PR as I don't think it's the correct approach, but please feel free to reopen if I'm missing something!

@anwather
Copy link
Author

Thanks @anthony-c-martin - tbh I haven't done it before :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Icon for bicep files in VS Code
4 participants