-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
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
Fixed odd chars on mac for agnoster theme #4001
Conversation
Fixed odd chars on mac for agnoster theme
Was this really necessary? I had Agnoster's theme working seamlessly on Mac before this change. Did you have a properly-patched powerline font set up? |
Hi, Yes, I have a properly-patched powerline font set up and I couldn't get it 2015-06-15 8:56 GMT+02:00 Nicolas Canceill [email protected]:
|
OK apparently this was reported in issue 2869. Anyway this should not break anything. |
|
||
# Fix odd char on mac | ||
if [[ `uname` == 'Darwin' ]]; then | ||
SEGMENT_SEPARATOR='\ue0b0' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We want ⮀and not \ue0b0 right? It doesn't work on my machine with a power line font.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How old is your Powerline font? \ue0b0
is the segment separator that Powerline currently uses. They changed the code points they were using a while back, and \ue0b0
is the new one. (I can't tell which other character you're referring to, because my browser isn't using a powerline-patched font. And since there are two versions of the font, pasting the characters directly as single Unicode characters leads to ambiguity, because their appearance is going to depend on which versions of the font people in the conversation have installed.)
We want \ue0b0 on Mac boxes and ⮀ on the rest...I have a Linux box too and the fix is working |
The \ue0b0 does not work on my Mac. I have the Menlo power line font. What do you suggest? |
I suspect it should actually be \ue0b0 everywhere, and this issue is caused by outdated Powerline fonts being installed (see #3713). AFAICT, Powerline doesn't use different Unicode characters on different OSes (which would be weird). But they did change the code points they were using back in 2012, so some machines may have old versions of the fonts installed. @ivanfoo: When you say "and ⮀ on the rest", I can't tell what character that is, because I don't use Powerline-patched fonts in my browser. (And if I did, it'd still be hard because the appearance would depend on which version I had installed.) What character (Unicode code point) are you referring to? What version (date or md5) of the patched font do you have installed on your Linux box? What's the behavior if you @ujjwalt: What exactly does "not work" mean? What are you seeing? Try re-downloading the Menlo Powerline font from https://github.com/powerline/fonts. If \ue0b0 does not work for you, it probably means you have an obsolete version (with old code points) of the font installed. To test which version of the font you're using (in your local terminal emulator), you can do this.
If you see a triangle "segment separator" followed by a branch symbol, you are using a current powerline-patched font. (I think.) If you see something else, you're looking at an old, non-patched, or otherwise bad font. E.g. here's what I see on my Mac, using Meslo LG M Regular ( I've put in #4091 to clarify this and make it consistent across platforms. |
So it does work if I use your font. |
That means your "Menlo for Powerline" font was an obsolete one with the old code points. I'd recommend switching to one of the "Meslo LG X for Powerline" variants. (The "X" just indicates different line spacing.) Meslo is a minor variant on Menlo with a few tweaks for on-screen use. You probably won't even notice the difference between them. |
Works perfectly now. You may close this issue. |
It's already closed. :) ("Merged" counts as closed.) |
Quick fix to solve odd separator char when using agnoster theme on Mac