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 Netwide Assembler (NASM) icon #857

Closed
3 tasks done
ignamartinoli opened this issue Jul 5, 2022 · 9 comments · Fixed by #990
Closed
3 tasks done

Add Netwide Assembler (NASM) icon #857

ignamartinoli opened this issue Jul 5, 2022 · 9 comments · Fixed by #990

Comments

@ignamartinoli
Copy link
Contributor

Requirements

  • I have searched the issues for my request and found nothing related and/or helpful
  • I have searched the FAQ for help
  • I have searched the Wiki for help

Is your feature request related to a problem? Please describe.

It would just be a nice add-on

Describe the solution you'd like

It would be cool to have an icon for Assembly languages for nvim-web-devicons and exa to use

Describe alternatives you've considered

I don't know any other solution

Additional context

There is no icon nor mascot for Assembly .asm files in general, but Netwide Assembler is quite popular and with its icon could fit that spot

Netwide_Assembler

@Finii
Copy link
Collaborator

Finii commented Jul 7, 2022

Related:

ogham/exa#1080

exa does not handle .asm at the moment at all: https://github.com/ogham/exa/blob/master/src/output/icons.rs

@Finii
Copy link
Collaborator

Finii commented Jul 7, 2022

lsd uses E614, which looks like a #

image

image

https://github.com/Peltoche/lsd/blob/master/src/icon.rs

That icon comes from Seti UI & Custom.

Well, I would not call # to be a good asm icon.
But this two balls, one big and a smaller top right, THAT would remind me too much of Lua (see glypg list above, the icon just below the #.

I see no consensus of assembler programmers for one specific icon, and the lack of a real icon for nasm does increase the impression they themselves do not count the two-balls thing relevant as 'this is asm' indicator.

The idea is good, but when I'm working with asm files and this icon turns up, I would think it a bug?

@ignamartinoli
Copy link
Contributor Author

I agree on that. I didn't thought about the similarity with Lua's icon, which is actually an embedded language so it wouldn't be too crazy to appear in a same codebase.

I can bring some better alternatives:

asm1-1

This one would be consistent with C and C++ ones and since the text is 3 letters only one would be able to distinguish it well enough

asm3

cpu

These two goes for a completely different route, suggesting the language operating on the CPU

What do you think about this ones?

@ignamartinoli
Copy link
Contributor Author

As an update, I was able to get this icon, which resembles the C/C++ but has a different shape orientation.
Does it seem like a good idea? Any feedback before creating a PR?

asm

@Finii
Copy link
Collaborator

Finii commented Oct 10, 2022

I must admit I think it looks quite good 😬

@ignamartinoli
Copy link
Contributor Author

Thanks @Finii! I will figure out how to create a PR and link the issue

@Finii
Copy link
Collaborator

Finii commented Oct 12, 2022

Some hints are in https://github.com/ryanoasis/nerd-fonts/blob/master/contributing.md#contributing-guide

The link in our documentation is broken I just noticed, it is https://egghead.io/courses/how-to-contribute-to-an-open-source-project-on-github

You need to turn your attention to the files in src/svgs/ in this repo. In principle... read that README, put your .svg there, and add the appropriate data to icons.tsv. After pushing the repo to github it will automatically rebuild src/glyphs/original.source.ttf with your icon included. Download that and check with fontforge or other font examination program.

(The font rebuild via github action can also be 'imitated' on your local machine, by running cd bin/scripts && ./generate-source-source.py, but that needs some stuff to be existing on your machine.) Do not put the generated font file into the PR.

:-) 👍

@Finii Finii mentioned this issue Oct 19, 2022
@ignamartinoli ignamartinoli mentioned this issue Nov 6, 2022
2 tasks
@ignamartinoli
Copy link
Contributor Author

@Finii I just opened a PR.

I'm not that experienced neither with Git nor this project so if there is any mistake please tell me so I can improve for my next contribution.

I wasn't able to use FontForge to verify, it would show me almost no icons.

Finii pushed a commit that referenced this issue Jan 13, 2023
[what]
Adds consistance with other low level languages icons like C/C++.

Reverted the change on `asm.svg`, added `asm_nf.svg`, modified `icons.tsv`

[note]
Closes #857.
Closes [990#pullrequestreview-1171459043](#990 (review))

Signed-off-by: igna_martinoli <ignamartinoli@protonmail.com>

[Note by Fini: Squashed two commits.]

Signed-off-by: Fini Jastrow <ulf.fini.jastrow@desy.de>
@github-actions
Copy link
Contributor

This issue has been automatically locked since there has not been any recent activity (i.e. last half year) after it was closed. It helps our maintainers focus on the active issues. If you have found a problem that seems similar, please open a new issue, complete the issue template with all the details necessary to reproduce, and mention this issue as reference.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 17, 2023
LNKLEO pushed a commit to LNKLEO/Nerd that referenced this issue Nov 24, 2023
[what]
Adds consistance with other low level languages icons like C/C++.

Reverted the change on `asm.svg`, added `asm_nf.svg`, modified `icons.tsv`

[note]
Closes ryanoasis#857.
Closes [990#pullrequestreview-1171459043](ryanoasis#990 (review))

Signed-off-by: igna_martinoli <ignamartinoli@protonmail.com>

[Note by Fini: Squashed two commits.]

Signed-off-by: Fini Jastrow <ulf.fini.jastrow@desy.de>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants