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

Icon nf-oct-file_directory changed on update to 3.0.0 #1207

Closed
mngshm opened this issue May 6, 2023 · 10 comments
Closed

Icon nf-oct-file_directory changed on update to 3.0.0 #1207

mngshm opened this issue May 6, 2023 · 10 comments

Comments

@mngshm
Copy link

mngshm commented May 6, 2023

Hey, I actually love an icon from the version 2.3 nerdfonts as due to the commit of modernizing the fonts, that glyh was uncertainly removed from it. One of my friend managed to get the file for that file from some other repo.

So, I copied that font to a different file and managed to make a generate a new font with only that icon in the file with the help of fontforge. But the issue is when i'm using a font from the v3.0 the icon is hollow and If I use the older version it is how I expect it to be.

Expectations (tested on font from v2.3) for v3.0
Sat May  6 11:39:49 IST 2023

This is how it is when using v3.0 fonts
Sat May  6 11:11:36 IST 2023

  • [x ] I have searched the issues for my issue and found nothing related and/or helpful
  • [ x] I have searched the FAQ for help
  • [ x] I have searched the Wiki for help

This is the symbol / glyph I want to be like expected above

My Setup

Arch Linux

@Finii
Copy link
Collaborator

Finii commented May 7, 2023

Is this about the directory/folder icon?

@mngshm
Copy link
Author

mngshm commented May 7, 2023

yes

@mngshm
Copy link
Author

mngshm commented May 7, 2023

also there's an endpoint in nerd fonts with the same endpoint as the one which I created in the separate file. So maybe the nerd font one is overlapping the one I made. So, is it that I have to create a new endpoint and shift this icon to that endpoint ?

@Finii
Copy link
Collaborator

Finii commented May 7, 2023

This in a duplicate of the comment here: #1190 (comment)

You seem to be missing the same glyph?

image

Octicons updated their design and a lot of their icons got modernized and unified in that process.

If it's just that it is 'open', maybe try one of their others:

image

Or enter folder on the cheat sheet and see if there is anything you can stand...

If you want to self patch and have a one-glyph font with the old icon and THAT does not render correctly, I could possibly help if I can get the hands on your font and font-patcher log.

@mngshm
Copy link
Author

mngshm commented May 7, 2023

Yes, I'm trying to fix this font for the guy who commented :D. could you please guide me on self patching ? I actually want the same glyph which is now turned into an unfilled or hollow icon.

@Finii
Copy link
Collaborator

Finii commented May 7, 2023

If you start with a already patched font and just want to patch that one glyph over from the single-glyph font.

The glyph you want to patch needs to be on the correct codepoint already (no movement possible).

Then do fontforge font-patcher --custom my_single_glyph_font.ttf --glyphdir path/to/the/font/file

I'm not sure what will happen to the font name, probably it will be broken :-|
It also depends on which font you started with, different fonts need different renaming setting.
Hmm, there is no 'no not change the naming' option.

So in principle what you would need to do is:

  • Start with the unpatched font(s)
  • Find out which makegroups mode it needs (src/unpatched-fonts/**/config.cfg)
  • Copy the single-glyph font into /src/glyphs/
  • Patch the fonts with -c --custom my_single_glyph_font.ttf --makegroups 4 (4 -> whatever is needed 1-6)
  • Of course add --mono or --propo on whatever the aim is

Or use the patch-em-all script, if you have it:

  • Copy the single-glyph font into /src/glyphs/
  • Patch the fonts with
    NERDFONTS='--custom my_single_glyph_font.ttf' ./gotta-patch-em-all-font-patcher\!.sh -cj /FiraMono
  • (FiraMono is the src/unpatched-fonts/ subdir name, adapt accordingly)
  • The NERDFONTS environment variable is used to transport more options through the patch-em-all
  • The makegroups will be automatically correct now

(Nothing tested, just from the top of my head)

@Finii
Copy link
Collaborator

Finii commented May 7, 2023

The -c option to gotta-patch-em puts the resulting font in check-fonts/ instead of patched-fonts/. Of course you do not need to give it, it just helps not messing up your repo ;-)

@Finii Finii changed the title Glyph from older version appears to be hollow Icon nf-oct-file_directory changed on update to 3.0.0 May 7, 2023
@mngshm
Copy link
Author

mngshm commented May 7, 2023

Thanks for all the help you''ve provided. I'll try this tomorrow morning as it's very late as of now. Also, please don't close this issue now itself ;)

@Finii
Copy link
Collaborator

Finii commented May 9, 2023

@siduck expressed the plan to re-package the modified fonts and have thus a different set up Nerd Fonts that are not Nerd Fonts. They should then create a fork and change the name, or whatever. But that plan will result in confusion.

Please do not do this.

@Finii Finii closed this as completed May 9, 2023
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 Nov 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants