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

Wrong highlighting in visual mode after latest update #960

Closed
antaed opened this issue Oct 18, 2019 · 14 comments
Closed

Wrong highlighting in visual mode after latest update #960

antaed opened this issue Oct 18, 2019 · 14 comments
Milestone

Comments

@antaed
Copy link

antaed commented Oct 18, 2019

I have set guifg=NONE for the Visual highlighting group in my colorscheme but it gets overridden after the latest update:

Screenshot 2019-10-18 at 11 10 40

@eirnym
Copy link
Contributor

eirnym commented Oct 18, 2019

@antaed Thank you for an issue,

Could you share your color scheme to test with?

@antaed
Copy link
Author

antaed commented Oct 18, 2019

Of course:
antaed.vim.zip

@eirnym
Copy link
Contributor

eirnym commented Oct 18, 2019

Thank you,

guifg/guibg is taken from Normal if it is NONE and visual fg/bg override syntax fg/bg respectively.

Could you tell me what effect you want to achieve?

@antaed
Copy link
Author

antaed commented Oct 18, 2019

I would like to have only the background highlighted in visual mode and preserve the syntax highlighting on the foreground, like this:
Annotation 2019-10-18 185908

(the screenshot above was taken on my Windows machine, the color scheme is exactly the same)

@eirnym
Copy link
Contributor

eirnym commented Oct 18, 2019

ViM in Terminal has the same issue and MacVim only paints that Vim said to paint.

You should report this issue to the upstream https://github.com/vim/vim as this issue definitely belongs there.

Sorry for misunderstanding on the beginning.

@antaed
Copy link
Author

antaed commented Oct 18, 2019

All right, I'll report the issue there. Thank you very much for your support, closing this right now.

@antaed antaed closed this as completed Oct 18, 2019
@eirnym
Copy link
Contributor

eirnym commented Oct 18, 2019

Please, report here the ticket url for tracking

@antaed
Copy link
Author

antaed commented Oct 18, 2019

Ok, I'll be back with the url. Do you happen to know which patch created the issue? It must be a very recent one as it worked fine until today.

@eirnym
Copy link
Contributor

eirnym commented Oct 18, 2019

I have snapshot 159, and have the same problem as you. Thank you for reporting!

@eirnym
Copy link
Contributor

eirnym commented Oct 18, 2019

@antaed the issue was fixed in the upstream, I hope it will be in snapshot 160, as many things were broken accidentally as I see in other issues there.

@antaed
Copy link
Author

antaed commented Oct 19, 2019

That was very quick, you're awesome. Thank you so much!

@ychin
Copy link
Member

ychin commented Oct 22, 2019

@eirnym What other issues are you looking to be fixed upstream?

@antaed
Copy link
Author

antaed commented Oct 22, 2019

Honestly, I haven't found any other issues besides this one. Thanks for asking

@ychin ychin added this to the snapshot-160 milestone Oct 22, 2019
@eirnym
Copy link
Contributor

eirnym commented Oct 22, 2019

@ychin as far as I see, other issues were fixed. It even could be built with Python 3.8 (tested it locally)

ychin added a commit to ychin/macvim that referenced this issue Oct 28, 2019
Vim patch 8.1.2224

General:

- MacVim **now requires macOS 10.9**, up from 10.8.
- MacVim binary is now notaraized by Apple. This should make it open
  properly without a security warning in macOS 10.15 Catalina. macvim-dev#959 macvim-dev#825

Features:

- Make mouse cursor higher contrast under macOS 10.13 or older. macvim-dev#910

Fixes:

- Fix OS appearance (dark vs light mode) detection not working. See
  `:help OSAppearanceChanged` for more info on the feature.  macvim-dev#973
- Fix spell checker flagging first words in line. macvim-dev#971 (Vim 8.1.2172)
- Fix wrong highlighting color scheme. macvim-dev#960 (Vim 8.1.2176)

Misc:

- Sparkle updater has been updated to 1.22.00. macvim-dev#963

Targets macOS 10.9+

Script interfaces have compatibility with these versions:

- Lua 5.3
- Perl 5.18
- Python2 2.7
- Python3 3.7
- Ruby 2.6
ychin added a commit that referenced this issue Oct 29, 2019
Vim patch 8.1.2224

General:

- MacVim **now requires macOS 10.9**, up from 10.8.
- MacVim binary is now notaraized by Apple. This should make it open
  properly without a security warning in macOS 10.15 Catalina. #959 #825

Features:

- Make mouse cursor higher contrast under macOS 10.13 or older. #910

Fixes:

- Fix OS appearance (dark vs light mode) detection not working. See
  `:help OSAppearanceChanged` for more info on the feature.  #973
- Fix spell checker flagging first words in line. #971 (Vim 8.1.2172)
- Fix wrong highlighting color scheme. #960 (Vim 8.1.2176)

Misc:

- Sparkle updater has been updated to 1.22.00. #963

Targets macOS 10.9+

Script interfaces have compatibility with these versions:

- Lua 5.3
- Perl 5.18
- Python2 2.7
- Python3 3.7
- Ruby 2.6
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

No branches or pull requests

3 participants