Skip to content

feat: add icons for Assembly files (*.S and *.asm) #530

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

Merged
merged 1 commit into from
Dec 24, 2024

Conversation

shizeeg
Copy link
Contributor

@shizeeg shizeeg commented Dec 23, 2024

No description provided.

@hasecilu
Copy link
Collaborator

hasecilu commented Dec 23, 2024

Hi, you need to move to the icons_by_file_extension table
image

Also, isn't better to use the same glyph? since it's text the ASM one seems to be easier to read, but it could be also a "chip" glyph to be more graphic.

@shizeeg
Copy link
Contributor Author

shizeeg commented Dec 23, 2024

oh, my bad. (fixed)
the thing is *.S files are GNU and *.asm files are usually nasm so they are not the same (kinda).

I'm not sure myself which one to use so I put them both. Which one would it be, then 'ASM' or the 'chip' one?

P.S. what about nvim-web-devicons.lua? Your guidelines say I have to add new icons to

local filetypes = {}

but I checked other commits, nobody seems to do that.

@hasecilu
Copy link
Collaborator

hasecilu commented Dec 23, 2024

Not sure how much changes between different architectures, personally I immediately thought of micrcontrollers.

image

P.S. what about nvim-web-devicons.lua? Your guidelines say I have to add new icons to

Not all types of files have a defined "filetype" on neovim, but yeah should be checked. In this case seems to be a asm.vim but not s.
I checked with ls /usr/share/nvim/runtime/ftplugin//ls /usr/share/nvim/runtime/syntax

image

@shizeeg
Copy link
Contributor Author

shizeeg commented Dec 24, 2024

both code.S (GNU Assembler) and code.asm (nasm, fasm, etc.) are recognized by NeoVim so I'm not sure what you meant. Since you mentioned 'ASM' icon is easier to read, I've replaced the 'chip' one with it.

P.S. The fae-chip icon is more fitted for smth like VHDL or other type of "hardware definition" language, IMHO.

@hasecilu
Copy link
Collaborator

You need to change the "S" to lowercase. Also add asm to filetype,

Regarding other filetypes I made a PR #531 to fix the other ones.

@shizeeg
Copy link
Contributor Author

shizeeg commented Dec 24, 2024

isn't it supposed to be uppercase? Here for example https://github.com/torvalds/linux/blob/master/arch/x86/boot/copy.S

I thought it's a weird gas naming convention. Whatever, I did as you said.
Good job with #531, btw. 👌

@hasecilu
Copy link
Collaborator

Yes, it should be uppercase but it wasn't working that way, now it works.

@hasecilu hasecilu merged commit c1a86a2 into nvim-tree:master Dec 24, 2024
4 checks passed
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.

2 participants