ryanoasis / nerd-fonts

Iconic font aggregator, collection, & patcher. 3,600+ icons, 50+ patched fonts: Hack, Source Code Pro, more. Glyph collections: Font Awesome, Material Design Icons, Octicons, & more
https://NerdFonts.com
Other
53.93k stars 3.63k forks source link

f482 went from symlink directory to bookmark slash? #1213

Closed duhdugg closed 1 year ago

duhdugg commented 1 year ago

🗹 Requirements

🎯 Subject of the issue

Experienced behavior:

f482 was previously showing a symlink directory icon, and now shows a bookmark slash icon

Expected behavior:

symlink icon

Example symbols:

🔧 Your Setup

★ Screenshots (Optional)

Before Upgrade (ttf-firacode-nerd-2.3.3-3)

2023-05-07-162650_61x31_scrot

After Upgrade (ttf-firacode-nerd-3.0.0-2)

2023-05-07-162710_65x28_scrot

Using the lsd command to generate the above screenshots, which shows this codepoint for symbolic link directories.

Related Issue

Submitted this issue to the octicons project as well: https://github.com/primer/octicons/issues/945

Edit Fini: Changed codepoint number

Finii commented 1 year ago

Sorry to hear that you lost a beloved icon with the update.

As you already found out the change is due to the Octicons update.

They dropped a lot icons; and the remaining folder icons changed to a more modern look.

That you see now the bookmark-slashed and not a .notdef (i.e. questionmark box) is because we reused the freed codepoints for newly added icons. The Octicons do not have native codepoints.

Unfortunately link-folder icons are not very widespread, so the only one left probably is this:

Screenshot 2023-05-08 at 06 42 48

But the 'folder' part looks quite different of course. Probably using that would mean to switch all folder related icons to Codicons.

Related is the general modernisation look-change of the folders, mentioned here https://github.com/ryanoasis/nerd-fonts/issues/1190#issuecomment-1532632029, with its related issue #1207.

Finii commented 1 year ago

Probably an Issue needs to be raised at lsd.

In principle we could of course keep the old style Octicons folder icons, but that would have to be with other codepoints, probably.

🤔 But that of course leads to maintenance hell... And if we do not follow updates of the sources, then we could stop updating at all. Hard.

Finii commented 1 year ago

I fear Octicons will not answer, and if they answer they will not change anything.

Can you create an Issue at lsd so that we can discuss together with them what to do?

Finii commented 1 year ago

If we come to a solution within the next days it could go into v3.0.1...

Finii commented 1 year ago

Maybe a good idea would be to add a symlink folder to the nf-cutom- folders, there are already some. But to have a unified look lsd would have to switch to that icons completely?

Screenshot 2023-05-08 at 07 01 25

Edit: Add image

Finii commented 1 year ago

Or I can "hand draw" a combination of the updated Octicons (normal) folder and a link-arrow.

Finii commented 1 year ago

https://github.com/lsd-rs/lsd/blob/2fe3fcdd3564836962eab8ba6b1444996fe24e1e/src/theme/icon.rs#L85

Did any of the other also change meaning, we need a list of dropped/changed codepoints

Finii commented 1 year ago
Icon in source font codepoint in Nerd Font
Missing octoface F008 F406
Missing keyboard F00D F40B
Missing gist F00E F40C
Missing file-text F011 F40E
Missing file-pdf F014 F411
Missing jersey F019 F416
Missing radio-tower F030 F424
Missing repo-force-push F04A F43E
Missing mail-reply F051 F443
Missing arrow-small-right F071 F45C
Missing gist-secret F08C F46C
Missing no-newline F09C F476
Missing arrow-small-up F09F F478
Missing arrow-small-down F0A0 F479
Missing arrow-small-left F0A1 F47A
Missing file-symlink-directory F0B1 F482
Missing circuit-board F0D6 F493
Missing watch F0E0 F49B
Missing text-size F0E3 F49E
Missing ellipses F101 F4A4
Missing plus-small F104 F4A7

Checking if they use more...

No. It is just the symlink-directory

$ grep -i 'f406|F40B|F40C|F40E|F411|F416|F424|F43E|F443|F45C|F46C|F476|F478|F479|F47A|F482|F493|F49B|F49E|F4A4|F4A7' ../../../icon.rs
            symlink_dir: "\u{f482}".into(),   // 
Finii commented 1 year ago

image

Finii commented 1 year ago

This is a 'combination' of file-symlink-file and file-directory; what do you think?

image

I will prepare everything to have that in 3.0.1 in the usual place.

siduck commented 1 year ago

hi @Finii could you add this icon? I can pay for it if you want

image

Finii commented 1 year ago

hi @Finii could you add this icon? I can pay for it if you want

I'm not doing this for money. I help for free if I can and feel that this project is respected.

duhdugg commented 1 year ago

I saw the latest update in my OS this weekend. It looks great. Thanks.

Finii commented 1 year ago

I saw the latest update in my OS this weekend. It looks great. Thanks.

Thank you :heart:

github-actions[bot] commented 10 months ago

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.