-
-
Notifications
You must be signed in to change notification settings - Fork 686
MacVim + Catalina Font issues #949
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
Comments
I actually loaded it into Photoshop... the pixels actually line up perfectly but they are just a tiny bit less transparent. I assume this isn't a MacVim issue, but some way OSX changed how they render fonts. I really wish I knew what it was because I absolutely hate it. All the fonts look naked and weak |
Ah! The line-height most definitely DID change though, thats what causes the most disturbing results.. there's so much space between the lines. I could live with the seemingly "weaker" letters, but the space between the lines is atrocious. I'm not sure how I would go about putting it back though. |
on different versions of macOS and in different apps I've used different fonts. This was also caused with different font versions. Currently I use |
I checked the version of Anonymous Pro but it seems to be the same for the last many years. It definitely changed though when upgrading to Catalina, and now I can't restore the old lineheight. :/ |
Could you test the font in different applications to check if they process it differently? Lastly MacVim started to honor more font properties on initialization, so it could affect as well. |
Oh wow... I just re-downloaded release 157 from the releases page and it was fixed (it was probably because i installed with --HEAD because of the other issue with homebrew hash not matching). So the new changes are definitely crazy on my end. Is that something that is 100% going to be merged into 158 and onwards? If so I will basically stop updating because the font is sadly unusable because it is hideous haha |
or is there any way to use a new version but keep the old font settings? |
Could you explain the difference? Screenshot difference will be the best. |
You can see it very clearly in the sidebar, it adds so much whitespace between the lines the second thing it does is very subtly makes the text 'crisper' but it also looks weaker. that is more subtle though and maybe isnt super clear in a photo, i used photoshop to check though. that is less of a big deal as the line height which totally kills my favorite font BEFORE (GOOD): AFTER (BAD): |
As a workaround I |
@9mm Thank you for sharing screenshots. You use modified I installed the font and tested it:
I'd select 13 which on my screen looks the best if I'd use this font. Anonymous Pro (original), size 12: |
@eirnym the font itself shouldn't have changed with the added powerline symbols. I tested here with both fonts, and you can use photoshop to verify they're identical, so the difference came from macvim/vim (installing old macvim with same powerline font works how it should) here is how it used to be for reference: @skyksandr thanks... i messed around with linespace and nothing actually changed before, and I just tried again this morning and it works! I probably mixed up set/let. This fixed it, thanks! Crazily enough, I had to do |
@9mm I've tried a few fonts from that repo (if we speak about the same mod repo), but fonts there in general seems to be very outdated for a few releases at least and some parameters were altered (like linespacing for some fonts). So yes, general symbols might not be changed, but other parameters and symbols might not be the same. Thus I've found a font with these symbols and which support a few languages I type with support of airline symbols (you see my last screenshot above). |
Sorry, maybe were not talking about the same thing. I was basically wondering what in macvim/vim changed for my fonts to be so different (it's not related to the 'with powerline' modification of the font because i've used the same font for 5 years or so). You may be correct that it modifies/ruins the font, but I haven't noticed any changes besides what is caused by the With all that said though, the |
I use the same build from GitHub (as I used on 10.14) and in my case nothing has been changed. For my font build from the Underneath MacVim uses |
Your font looks better with |
Thanks it fixed it, and now I removed linespace=-3 👍 |
Good to hear that! |
Recent change to stateful renderer (macvim-dev#858) has inadvertantly changed how MacVim handles line spacing. Previously, MacVim intentionally ignores the line spacing of a font and creates a new dummy font that essentially has line spacing of 1, but the new code uses the font as is. This means font with non-standard line spacing (e.g. Input Mono) will look different. This is technically the correct way to handle fonts but is different from how MacVim has worked for years. Also, see last time this regression (where MacVim didn't discard line spacing) happened in macvim-dev#928 / macvim-dev#949 which was fixed in macvim-dev#957. Also see macvim-dev#977 where the bug was filed the other way requesting for using the font's line spacing instead of discarding it. This commit re-introduces the behavior to discard line spacing, but only provides it as an option (can be set in the preference pane), while defaulting to using the line spacing as that seems more correct. Note that from a casual survey of other terminals and editors, this behavior is quite inconsistent. Xcode does use the font's line spacing, and was partially the motivation of switching to that as a default. Close macvim-dev#1152.
Describe the bug
Before installing Catalina, MacVim font looked fantastic when paired with
Anonymous Pro
(https://www.marksimonson.com/fonts/view/anonymous-pro).Now however, everything looks very weird, like it's spaced out with extra pixels between each letter.
Here is before installing (I believe this was Mojave when I took this screenshot):
and here is after
Environment (please complete the following information):
vim --version
.) latest mvim from homebrew --HEADThe text was updated successfully, but these errors were encountered: