Mozart409 / mozart409-icons

My personal choice of existing themes combined into one. Based on Ayu Icons with my own modifications and additions.
https://marketplace.visualstudio.com/items?itemName=Mozart409.mozart409-icons
MIT License
1 stars 0 forks source link

Nim lang icon request :) #4

Open Nuc1eoN opened 2 years ago

Nuc1eoN commented 2 years ago

Hi, @Mozart409 I use your theme in my main setup and I love it! Thank you!

The only thing that is missing for me is an icon for nimlang files. Nim is an awesome language check it out here https://nim-lang.org/

The corresponding file extensions are: .nim, .nims (nimscript), and .nimble (package manager). [EDIT: There's also .nim.cfg]

For reference here are the icons in material theme: image

Thank you in advance and keep up the good spirit! <3

Nuc1eoN commented 2 years ago

vscode-icons actually used a different icon for .nimble image A yellow box with the nim crown logo.

Mozart409 commented 2 years ago

Thank you for your request. I will get too it 👍🏻

Mozart409 commented 2 years ago

@Nuc1eoN I released a new version with those icons. Tell me if you like them and if everything works for you. 👍🏻

Nuc1eoN commented 2 years ago

@Mozart409 Wooww thank you I think it looks awesome! Better then the other versions that I saw <3

Only thing is, it strangely does not apply to *.nimble.json and *.nim.cfg on my side? Does it work you?

Mozart409 commented 2 years ago

Nice. 👍🏻

Oh I haven't read the docs. So I though those were special files, like "Dockerfile" and not file extensions. I can push a fix in 10 min. Hold on.

Nuc1eoN commented 2 years ago

One sec I am checking rn how those files apply

Nuc1eoN commented 2 years ago

I didn't have those two in my projects yet but I will check

Nuc1eoN commented 2 years ago

Ok so as far as I gathered there are the file extensions: *.nim.cfg, *.nimble-link

There is also a nimblemeta.json file.

I don't think there is a .nimble.json though, where have you got that from?

Mozart409 commented 2 years ago

So I wanted to add the icons for *.nimble.json and I'm having problems. I dont have the time right now to fix it, so it will need to wait until the weekend. Im sorry.

Mozart409 commented 2 years ago

Ok so as far as I gathered there are the file extensions: *.nim.cfg, *.nimble-link

There is also a nimblemeta.json file.

I don't think there is a .nimble.json though, where have you got that from?

There isnt a .nimble.json. I though you ment a file named .nimble.json and not *.nimble.json. My bad

Nuc1eoN commented 2 years ago

There isnt a .nimble.json. I though you ment a file named .nimble.json and not *.nimble.json. My bad

I do not think there is either of them . I think this definition can be removed entirely.

EDIT:

Ok so as far as I gathered there are the file extensions: .nim.cfg, .nimble-link

There is also a nimblemeta.json file.

^only this :D

Mozart409 commented 2 years ago

Ahh ok. So tell me if i forgot any file extensions

.nimble-link nimblemeta.json .nim .nims .nimble

// needs to be added nimdoc.cfg

// find a way to only match .nim.cfg and not .cfg *.nim.cfg

Nuc1eoN commented 2 years ago

.nim.cfg .nimble-link nimblemeta.json .nim .nims .nimble

Yes that is correct! Only last one should be *.nimble (it's a file extension too).

Technically yes, everything with "nimble" in the name is related to the package manager, so it could have like it's own icon. But I am also fine with how it is, much better than before! :D

Mozart409 commented 2 years ago

Ok perfect. I updated my comment. I have everything ready except "*.nim.cfg"

Nuc1eoN commented 2 years ago

What is the matter with "*.nim.cfg"?

Mozart409 commented 2 years ago

there is a conflict between ".cfg" and ".nim.cfg"

".nim.cfg" matches to ".cfg" which has a different icon. I bet there is a way around it, but I don't know how, and I don't have the time right now.

I will push the changes now and fix the last one Saturday or Sunday

Nuc1eoN commented 2 years ago

Oh god dammit! Now somebody commented on nim irc there is also nimdoc.cfg :laughing:

Sorrry grrr

Mozart409 commented 2 years ago

Damn. Ok I will add that to the list and fix it. The update should be live.

Nuc1eoN commented 2 years ago

Thank you and no hurry with the other fix! Take your time and have a good weekend! :smile:

Thank you for the work. If you make a donate button I send you a small amount :D

Mozart409 commented 2 years ago

No, thank you. You too 😄

What really? Thank you. I will add one. During the weekend.