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

fixed viml function background #102

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ialsaffar
Copy link
Contributor

@ialsaffar ialsaffar commented Jan 26, 2021

I noticed that vim script function names have different backgrounds in certain situations. For example, if you turn on cursor line with :set cursorline, vim function names will have default dark background rather than cursorline background. I don't think this is the intended behavior. Correct me if I'm wrong.

This should be easy to reproduce. Please let me know if you can't.

Also, yesterday vim functions on my system had black background. Unfortunately, I wasn't able to reproduce it today.

This PR links the vim function highlight group to the normal function group. That way, by defining function group colors, vim functions "inherit" the same definitions. Does this sound reasonable?

This fix was inspired by looking at nord's nord.vim.

@ialsaffar
Copy link
Contributor Author

Also, not sure if this needs to be done for functions of other languages.

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.

1 participant