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

Update VSCode Icons #6115

Closed
vince-fugnitto opened this issue Sep 5, 2019 · 9 comments · Fixed by #9864
Closed

Update VSCode Icons #6115

vince-fugnitto opened this issue Sep 5, 2019 · 9 comments · Fixed by #9864
Labels
enhancement issues that are enhancements to current functionality - nice to haves help wanted issues meant to be picked up, require help monaco issues related to monaco vscode issues related to VSCode compatibility

Comments

@vince-fugnitto
Copy link
Member

Description

The repository vscode-icons has recently been created where we can re-use the same icons present in VSCode. Is it something we'd like to align to? If we use the outdated icons it's quite possible we will get a mismatch in our own applications when using plugins (which bring the new icons in their views).

@vince-fugnitto vince-fugnitto added question user / developer questions vscode issues related to VSCode compatibility proposal feature proposals (potential future features) labels Sep 5, 2019
@akosyakov
Copy link
Member

akosyakov commented Sep 6, 2019

New Monaco already using new VS Code icons, so probably it makes sense to align.

@JesterOrNot
Copy link

When would something like this come to pass?

@akosyakov akosyakov added enhancement issues that are enhancements to current functionality - nice to haves help wanted issues meant to be picked up, require help and removed proposal feature proposals (potential future features) question user / developer questions labels Jan 20, 2020
@akosyakov
Copy link
Member

@JesterOrNot there was not yet plans to work on it. Does anyone ask about it for Gitpod?

@JesterOrNot
Copy link

No but I think it would be pretty cool 😅

@akosyakov
Copy link
Member

akosyakov commented Jan 21, 2020

It seems that Monaco comes with icons already, ideally it would be good to use it from monaco distribution then that editor and IDE icons are aligned. It's a tricky and for the meantime just updating icon files could be enough. @JesterOrNot looking into it

@JesterOrNot JesterOrNot mentioned this issue Jan 29, 2020
88 tasks
@akosyakov akosyakov added the monaco issues related to monaco label Mar 18, 2020
@akosyakov
Copy link
Member

After last Monaco upgrade we have codeicons in our css. So it should be a matter of going and changing css and then removing icon files. Please see for details on how to use codeicons: #4835 (comment)

We should also figure out how to deprecate old icons. They are probably used in end products.

@vince-fugnitto
Copy link
Member Author

vince-fugnitto commented Apr 2, 2020

After last Monaco upgrade we have codeicons in our css. So it should be a matter of going and changing css and then removing icon files. Please see for details on how to use codeicons: #4835 (comment)

We should also figure out how to deprecate old icons. They are probably used in end products.

I tried to update an existing font-awesome icon into a codicon but it did not seem to work properly. I'm wondering if the icons are only available for monaco components (quick-open) at the moment, and some extra steps are required to use them across the application.

In order to use codicons in other areas of the application, the following method is required:

codicon codicon-{codicon-name}

Ex: codicon codicon-folder-open

@akosyakov
Copy link
Member

It would mean though that icons won't work without Monaco extension :( We could start like that and if someone comes with the issue later, we will need to provide such css classes through the core then.

@vince-fugnitto
Copy link
Member Author

It would mean though that icons won't work without Monaco extension :( We could start like that and if someone comes with the issue later, we will need to provide such css classes through the core then.

I think we can go ahead for now with these icons (since it is quite difficult not to use monaco), and if people experience issues we can somehow implement fallback icons: #5009.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement issues that are enhancements to current functionality - nice to haves help wanted issues meant to be picked up, require help monaco issues related to monaco vscode issues related to VSCode compatibility
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants