-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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 octicons and drop old MDI #1156
Conversation
Current output:
I guess the icons that have been dropped are no problem (but then, what do I know what people use these icons for). |
6dc35a2
to
8cbb6de
Compare
@jef @djensenius What do you think, should we use the 16px or the 24 px octicons? |
673cf75
to
c8fa239
Compare
Fixed copyright issue, force push (sorry for the frequent pushes) |
We need to wait until Material Design Icons moved out of the way, because we need their old (dropped with v3.0.0) codepoints partially ( We just need to do |
Great point. IIRC, the 24px octicons have some that aren't available in 16px or vice versa. If not, then 16px sounds great! |
Amazing! Nice work and thanks for all the updates :) |
[why] The octicons got a lot updates. But they do not have a font anymore. [how] Keep our old codepoints constant, but add the new icons thereafter. This commit just moves all the mechanics in and moves the (old) font. No actual update here. The mapping file has been created with the analyze_octicons script. Fixes: #490 Signed-off-by: Fini Jastrow <[email protected]>
This commit does the actual update of the octicons.ttf font file. The script to update is 'generate'. It uses the mapping file from the previous commit to create a new compatible font. Some icons have meanwhile been dropped. We use their codepoints for new icons. Also fix a little bug in the GlyphInfo writer. Following the output of the actual script run. $ ./generate feature/update-octicons ● 1 … 5 ⚑ 5 Reading mapping file Found 172 entries Fetching octicons archive "v18.3.0.tar.gz" % Total % Received % Xferd Average Speed Time Time Time Current Dload Upload Total Spent Left Speed 0 0 0 0 0 0 0 0 --:--:-- --:--:-- --:--:-- 0 100 550k 0 550k 0 0 504k 0 --:--:-- 0:00:01 --:--:-- 919k Unpacking octicons archive Found 576 svgs Found 309 icons after de-duplicating Renamed cloud-download -> download Renamed cloud-upload -> upload Renamed clippy -> paste Renamed mail-read -> read Renamed primitive-dot -> dot-fill Renamed primitive-square -> square-fill Renamed settings -> sliders Renamed dashboard -> meter Renamed trashcan -> trash Renamed paintcan -> paintbrush Missing octoface Missing keyboard Missing gist Missing file-text Missing file-pdf Missing jersey Missing radio-tower Missing repo-force-push Missing mail-reply Missing arrow-small-right Missing gist-secret Missing no-newline Missing arrow-small-up Missing arrow-small-down Missing arrow-small-left Missing file-symlink-directory Missing circuit-board Missing watch Missing text-size Missing ellipses Missing plus-small Found 151 (of 172, missing 21) and new 158 Filled in missing, remaining new 137 Appended remaining new, total new mapping 309 Generating octicons.ttf with 309 glyphs Generating GlyphInfo i_oct.sh Finished Signed-off-by: Fini Jastrow <[email protected]>
[why] The old MDI are kept, just disabled. Maybe we need them or someone else, for backward compatibility. Can be dropped sometime in the future. Signed-off-by: Fini Jastrow <[email protected]>
c8fa239
to
e6e2058
Compare
This can only be done after MDI-old has been dropped
[why]
The octicons got a lot updates.
But they do not have a font anymore.
[how]
Keep our old codepoints constant, but add the new icons thereafter.
Fixes: #490
Requirements / Checklist
What does this Pull Request (PR) do?
Update the Octicons icons.
If one icon has been dropped (without replacement) re-use its codepoint for a newly added icon.
Append other newly added icons in the end.
Afterwards Octicons will be gapless in
F400
-F532
in Nerd Fonts.Note that we reassign codepoints on patching. The original octicons font has many gaps.
To keep also compatible here the font we create here is a bit ... stranger, with a lot of gaps and some gaps filled.
The methodology seems complicated by the goal is easy: We just want to patch in the icon into Nerd Fonts with the aformentioned codepoints and no additional special rules.
This means for the
octicons.ttf
Existing gaps are preserved.
Dropped icons are replaced by newly added ones.
The remainder of new icons is added in the back, which is FAR back, because of the one stray icon the original (old) font had.
How should this be manually tested?
Any background context you can provide?
What are the relevant tickets (if any)?
Screenshots (if appropriate or helpful)