gorbit99 / codewindow.nvim

MIT License
440 stars 17 forks source link

[Feature] Allow configuring the window's appearance #19

Closed akinsho closed 2 years ago

akinsho commented 2 years ago

It would be nice if the user could change some simple things about the window, e.g.

image

For example, in my case I change the border of all floating windows by all plugins to match my desired custom window border and tweak the highlights to match, but currently this isn't possible. Also, as you can see in the screengrab the signcolumn is coloured which is probably because I highlight the background of diagnostic signs but not the sign column itself, but the highlights are probably taken from the signs themselves

gorbit99 commented 2 years ago

For example, in my case I change the border of all floating windows by all plugins to match my desired custom window border and tweak the highlights to match, but currently this isn't possible.

For now you can work around this by defining a highlight named CodewindowBorder before you open the minimap (so probably before setup).

but the highlights are probably taken from the signs themselves

Indeed, they are just DiagnosticSignWarn and DiagnosticSignError. I'll probably put in highlight groups that by default link to these two, but can be overwritten.

Edit: I think you meant you want to change the border pattern. I might make a config that just overrides the default params passed to windows.

akinsho commented 2 years ago

Sounds great @gorbit99, tbh I meant the highlight specifically, but both are good since generally I change the border of all my floating windows to a custom style and since it's not currently possible to do this at the vim level it's good to have the option in a plugin

gorbit99 commented 2 years ago

Added in https://github.com/gorbit99/codewindow.nvim/commit/b412f022df7282ef91d922385c62c58bb9b03d0e

You can change the highlights by defining the highlight groups CodewindowBackground, CodewindowBorder, CodewindowWarn and CodewindowError, and you can change the border style with the window_border config option

akinsho commented 2 years ago

@gorbit99 I just tested this out and I don't think it works for the code window border because that highlight is used for VertSplit in the winhighlight not for the FloatBorder. I think VertSplit should be changed to FloatBorder since atm the CodewindowBorder highlight does not affect the border highlight

gorbit99 commented 2 years ago

@gorbit99 I just tested this out and I don't think it works for the code window border because that highlight is used for VertSplit in the winhighlight not for the FloatBorder. I think VertSplit should be changed to FloatBorder since atm the CodewindowBorder highlight does not affect the border highlight

Ah, actually, I did test it and it worked, but it's probably because FloatBorder (in my case) is linked to VertSplit. I'll change it to that in a sec.

gorbit99 commented 2 years ago

Solved in https://github.com/gorbit99/codewindow.nvim/commit/7123d6fb4dc24b0433d8a9b37287f3b5e9a20d2c

akinsho commented 2 years ago

Awesome that works great now 👍🏿

Integralist commented 2 years ago

👋🏻 Sorry I wasn't able to follow along that well, but I didn't want to open a separate issue just in case my issue was related.

In the below screenshot you can see that some dots have highlights (at the top some are green, a little bit below that there are some orange and blue dots) and I'm not sure why?

Screenshot 2022-10-26 at 09 27 21

In other files this can be a bit more distracting...

Screenshot 2022-10-26 at 09 35 06

Now at first, I thought maybe the dots were being coloured somehow in relation to the code syntax highlighting but now I think this actually has something to do with highlight groups (as I'm guessing the dots are using some kind of diagnostic 'sign' symbol maybe).

Is this the case? How can I get rid of these additional colours if they are indeed meaningless in relation to what minimap is displaying (don't want to break any diagnostic stuff that my current buffer's LSP might be using for example).

Thanks!

gorbit99 commented 2 years ago

@Integralist Please open a new issue, it's better to close that in case it is related, than dealing with separate issues in the same bugreport