Skip to content

Add eza icons #164

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

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

Conversation

supersurviveur
Copy link

@supersurviveur supersurviveur commented Apr 17, 2025

Hi !
This PR add some icons mainly used by eza, and also some that I see in my terminal sometimes.
Here are the changes :
image

  • spotify
  • Readme.md
  • download folder
  • key folder
  • music folder
  • makefile
  • ocaml
  • go
  • yarn
  • bash
  • vector curve (shift left)
  • Hex/Binary file
  • Translation
  • Outline key
  • ssh
  • toml

Icons I copied because they have multiple unicode code in nerd fonts :

  • docker
  • rust

@the-moonwitch
Copy link
Owner

I checked a number of these and they don't follow the correct sizing for Cozette glyphs. It's also additionally difficult to review the changes since you didn't include codepoints or the source icon set. I can't merge this in its current state.

@supersurviveur
Copy link
Author

Thanks, I don't have the time now but I will try to write the exact codepoints.

I don't understand clearly which icons are not correctly sized, there are already some icons in the font a little bit bigger than 6x13.

I can remove problematic icons if needed, or try to reduce their size but it seems difficult.

@supersurviveur
Copy link
Author

In the order of the original screenshot :

  • U+F1BC: 
  • U+F00BA: 󰂺
  • U+F08AC: 󰢬
  • U+F024D: 󰉍
  • U+F1359: 󱍙
  • U+E673: 
  • U+E67A: 
  • U+E65E: 
  • U+E6A7: 
  • U+F1183: 󱆃
  • U+F0559: 󰕙
  • U+EAE8: 
  • U+F05CA: 󰗊
  • U+F0DD6: 󰷖
  • U+F08C0: 󰣀
  • U+E6B2: 

And the two duplicated icons :

  • docker: U+E650
  • rust: U+E68b

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