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 icon: nginx (original, plain) #2291

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

hasecilu
Copy link

@hasecilu hasecilu commented Oct 28, 2024

Double check these details before you open a PR

  • PR does not match another non-stale PR currently opened

Features

[why]
The aspect ratio of the previous nginx logo made it very hard to read/identify on terminals, this new icon have a better aspect ratio making it identifiable.

[how]
Rename previous icon as wormark, add new original icon and its plain version to make it suitable for fonts, specifically Nerd Fonts.

This PR closes NONE

Notes

Ideally the new icon will be use for the next Nerd Fonts version.

nginx-original

[why]
The aspect ratio of the previous nginx logo made it very hard to
read/identify on terminals, this new icon have a better aspect ratio
making it identifiable.

[how]
Rename previous icon as wormark, add new original icon and its plain
version to make it suitable for fonts, specifically Nerd Fonts.
Finii added a commit to ryanoasis/nerd-fonts that referenced this pull request Nov 13, 2024
[why]
Plain icon is missing but would be better than the wordmark icon.

[how]
Add plain icon, taken from their website.

[note]
Similar (but slightly different svg) to
devicons/devicon#2291

Suggested-by: hasecilu
Signed-off-by: Fini Jastrow <[email protected]>
Finii added a commit to ryanoasis/nerd-fonts that referenced this pull request Nov 13, 2024
[why]
Plain icon is missing but would be better than the wordmark icon.

[how]
Add plain icon, taken from their website.

[note]
Similar (but slightly different svg) to
devicons/devicon#2291

Suggested-by: hasecilu
Signed-off-by: Fini Jastrow <[email protected]>
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.

1 participant