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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Highlight group for EndOfBuffer #23

Closed
akinsho opened this issue May 6, 2021 · 8 comments
Closed

Highlight group for EndOfBuffer #23

akinsho opened this issue May 6, 2021 · 8 comments

Comments

@akinsho
Copy link

akinsho commented May 6, 2021

Hi 馃憢馃従 ,

I just started customising my LspTroubleNormal background and noticed that it doesn't include the EndOfBuffer highlight so things looks like this.

image

The first line is the darkened highlight and the bottom is the EndOfBuffer. I think it should default link to LspTroubleNormal so it would be something like.

setlocal winhighlight=Normal:LspTroubleNormal,EndOfBuffer:LspTroubleNormal,SignColumn:LspTroubleNormal

This way a user can change the color of the full window not just part of it.

@folke
Copy link
Owner

folke commented May 6, 2021

Are you sure this is the issue here?

If I set the bg to red, the full window gets the bg color.

image

@folke
Copy link
Owner

folke commented May 6, 2021

You probably changed the LspTroubleText to a different bg than LspTroubleNormal?

@folke
Copy link
Owner

folke commented May 6, 2021

That's probably not it. Is it possible something else has changed the winhighlight option for the trouble window?

What does setlocal winhilight? give?

setlocal winhighlight=Normal:LspTroubleNormal should be enough.

@folke
Copy link
Owner

folke commented May 6, 2021

Nevermind. You're right. My theme doesn't set a bg for EndOfBuffer, so for me it's working.

I'll fix it. Thanks!

@folke
Copy link
Owner

folke commented May 6, 2021

it doesn't just work like that, since then the ~ at the end of buffer will get a foreground color, making it worse.

@folke
Copy link
Owner

folke commented May 6, 2021

Can fix that too with setlocal fcs=eob:\ :)

@folke
Copy link
Owner

folke commented May 6, 2021

Fixed. Can you check?

@akinsho
Copy link
Author

akinsho commented May 6, 2021

@folke awesome 馃憤馃従 yep that's fixed it 馃殌

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

2 participants