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

Add icon support #26

Closed
fsmaia opened this issue Jun 28, 2019 · 6 comments
Closed

Add icon support #26

fsmaia opened this issue Jun 28, 2019 · 6 comments

Comments

@fsmaia
Copy link

fsmaia commented Jun 28, 2019

I just added our jenkinsfile alias to vscode-icons repository:
vscode-icons/vscode-icons#2116

I'm starting this issue just for discussion if changes are required in this repository (e.g. renaming the alias to jenkins).

Hope it gets merged soon 🎉

@ghost
Copy link

ghost commented Jun 30, 2019

I'm curious about what would happen if we changed the alias to "jenkins" in this repo and someone was using an icon theme other than vscode-icons, like Monokai Pro for instance. Would their Jenkinsfile support just go away? I'm sure we could add backward compatibility if that's the case.

@fsmaia
Copy link
Author

fsmaia commented Jul 4, 2019

Hmm 🤔
What's the most straightforward alias for Jenkinsfile? Just couldn't find out Monokai Pro icon names. If it is jenkinsfile so we could keep our current name, as we expect a merge in vscode-icons.

@ghost
Copy link

ghost commented Jul 8, 2019

In hindsight, I think jenkins is more than sufficient. I'm not sure if any code changes here are necessary. Perhaps someone else could shed some light on this.

@sgwozdz
Copy link
Owner

sgwozdz commented Jul 23, 2019

TL;DR
If you want to have icons for your jenkins files, you have to add it to the icon theme you're using.

I'm curious about what would happen if we changed the alias to "jenkins" in this repo and someone was using an icon theme other than vscode-icons, like Monokai Pro for instance. Would their Jenkinsfile support just go away? I'm sure we could add backward compatibility if that's the case.

Icon themes and extensions to support languages are completely separate things. But both of them decide when to apply its logic based on the same rule - file name/extension. So in theory, if you add something here:

 "extensions": [
     "Jenkinsfile",
     "jenkinsfile",
     ".jdp"
 ],
 "filenames": [
     "Jenkinsfile",
     "jenkinsfile"
 ]

User will see this extension features in his file with the same extension or name.
I've added similar to the theme I'm using - PKief/vscode-material-icon-theme#502

@sgwozdz
Copy link
Owner

sgwozdz commented Aug 7, 2019

@fsmaia your change to vscode-icons seems to deployed, can we close the ticket?

@fsmaia
Copy link
Author

fsmaia commented Aug 7, 2019

For sure! Big thanks :)

@sgwozdz sgwozdz closed this as completed Aug 7, 2019
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

No branches or pull requests

2 participants