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

Add powerline git, LN, lock symbols #5054

Merged
merged 1 commit into from
Jun 3, 2024
Merged

Conversation

joelhockey
Copy link

Symbols taken from https://github.com/powerline/fontpatcher/blob/develop/fonts/powerline-symbols.sfd

The original symbols use height and widith of approx 1060x2048, so I modified to be roughly 1000x1000 via:

fontforge -lang=ff -c 'Open($1); SelectAll(); UnlinkReference(); Scale(100, 50); Move(0, 300); Export("svg/%n-%e.svg");' powerline-symbols.sfd

Then I edited by hand to round values to approx 2 significant digits, and then used a script to convert relative SVG commands such as 'l', 'c', 's' to use only 'M', 'L' and 'C' and used expected format with comma separators rather than spaces, and divide all values by 1000 to map into a 1x1 space.

Symbols taken from https://github.com/powerline/fontpatcher/blob/develop/fonts/powerline-symbols.sfd

The original symbols use height and widith of approx 1060x2048, so I
modified to be roughly 1000x1000 via:

fontforge -lang=ff -c 'Open($1); SelectAll(); UnlinkReference(); Scale(100, 50); Move(0, 300); Export("svg/%n-%e.svg");' powerline-symbols.sfd

Then I edited by hand to round values to approx 2 significant digits,
and then used a script to convert relative SVG commands such as 'l',
'c', 's' to use only 'M', 'L' and 'C' and used expected format with
comma separators rather than spaces, and divide all values by 1000 to
map into a 1x1 space.
@joelhockey
Copy link
Author

@Tyriar or @jerch or anyone, is there a better way to request a review for a pull request?

@joelhockey
Copy link
Author

@meganrogge @sbatten who have committed to CustomGlyph.ts in the past

@jerch
Copy link
Member

jerch commented Jun 3, 2024

@joelhockey Imho @Tyriar wrote the custom glyph handling code, so he might be best to review it.

@Tyriar Tyriar added this to the 5.6.0 milestone Jun 3, 2024
@Tyriar Tyriar self-assigned this Jun 3, 2024
Copy link
Member

@Tyriar Tyriar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is fantastic! 👏

image

Zoomed:

image

Big letter spacing and line height:

image

@Tyriar Tyriar enabled auto-merge June 3, 2024 14:31
@Tyriar Tyriar merged commit baa4e1b into xtermjs:master Jun 3, 2024
16 checks passed
@joelhockey
Copy link
Author

Thanks @Tyriar, would you also be able to review #4879 by @JasonXJ. We both work on ChromeOS Terminal app.

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.

None yet

3 participants