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

Retain glyph names when merging #41

Closed
eugenesvk opened this issue Nov 8, 2015 · 4 comments
Closed

Retain glyph names when merging #41

eugenesvk opened this issue Nov 8, 2015 · 4 comments
Assignees
Labels
Milestone

Comments

@eugenesvk
Copy link

I've noticed that glyph names are lost when patching fonts. For example, if I take the Octicons font as a source of glyphs and patch another font with it (with the --octicons option), then all the glyph names (e.g. octoface or mark-github or repo-forked for some of the standard GitHub glyphs) are lost and instead replaced by Unicode glyph codes (e.g. uniF008 for octoface).

Is there a chance to retain those names? They are very useful in finding specific glyph codes (remembering a name is much easier than remembering a code, especially since codes can change between font versions, while name is usually more stable).
Thank you!

@ryanoasis
Copy link
Owner

@eugenesvk

I think I have found a solution to this 😄, I will include this with the next release if it tests out well. Thanks for reporting this 👍

somewhat related (in that info is lost): #43

@ryanoasis ryanoasis self-assigned this Nov 9, 2015
@ryanoasis ryanoasis added this to the v0.6.0 milestone Nov 9, 2015
@eugenesvk
Copy link
Author

Ok, thanks a lot! Will be looking forward to it.
By the way, I've noticed another thing - some of the glyphs in the end of Octicons are not appended (e.g. F0E0 for 'watch' or F27C for 'device-desktop). I've downloaded the latest version before embedding (which has a few extra glyphs), so my guess would be that the address range is not updated yet. If that is the case, would you kindly update the range to for the --octicons option so that all glyphs get embedded?
Thanks!

P.S.
Never used github before, so not sure whether I should close this issue myself or you will do it after you release an update.

@ryanoasis
Copy link
Owner

@eugenesvk

Yes you are correct Octicons I am using is not up to date, see: #34 (comment)

No worries I was actually late to the GitHub party as well 😄 . For this issue we can leave it open and I will close it on the next release.

@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 Mar 13, 2023
LNKLEO pushed a commit to LNKLEO/Nerd that referenced this issue Nov 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants