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

replace glyphs in material-nf which are overwritten arabic chars #1280

Merged
merged 1 commit into from
Jul 21, 2021

Conversation

freswa
Copy link
Contributor

@freswa freswa commented Jul 21, 2021

NerdFonts has one big issue. They overwrite code points which are
reserved for chinese and arabic letters (range 0xf500-0xfd46).
While overwriting chinese letters is bad, they are LTR glyphs and print
"normally".
net_loopback and update used to be symbols that usually map
to arabic letters which make pango print those strings RTL. I
tried to fix this with the LTR marker 0x200e in #1206 but it failed.
This PR replaces the symbols with similar ones that don't map to
the arabic charset.
nerd-font issue: ryanoasis/nerd-fonts#365
fixes #1206

NerdFonts has one big issue. They overwrite code points which are
reserved for chinese and arabic letters (range 0xf500-0xfd46).
While overwriting chinese letters is bad, they are LTR glyphs and print
"normally".
`net_loopback` and `update` used to be symbols that usually map
to arabic letters which make pango print those strings RTL. I
tried to fix this with the LTR marker 0x200e in greshake#1206 but it failed.
This PR replaces the symbols with similar ones that don't map to
the arabic charset.
nerd-font issue: ryanoasis/nerd-fonts#365
fixes greshake#1206
@ammgws
Copy link
Collaborator

ammgws commented Jul 21, 2021

Thanks! This is the best approach so far since it has been two years and #365 still has not been addressed.

@ammgws ammgws merged commit 3dad90f into greshake:master Jul 21, 2021
@freswa freswa deleted the fix-arabic-nf-symbols branch July 21, 2021 23:46
@MaxVerevkin
Copy link
Collaborator

Thanks! This is the best approach so far since it has been two years and #365 still has not been addressed.

@ammgws How does #365 relate it this?

@GladOSkar
Copy link
Contributor

GladOSkar commented Jul 22, 2021

I assume he means ryanoasis/nerd-fonts#365

MaxVerevkin pushed a commit to MaxVerevkin/swaystatus that referenced this pull request Jul 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

material-nf: net block is printed in reverse order when loopback device is used
4 participants