-
Notifications
You must be signed in to change notification settings - Fork 0
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
Make highlights more configurable #4
Comments
Leaving it off by default sounds like a safe bet, just having the option in the config would be great. Do you want me to try implementing this or would you do it? |
I'm still not sure exactly what mechanism we should use yet. Personally, I lean toward having a section in the config like: orgmode = {
highlight_groups = {
category = "Variable",
headline1 = "etc",
}
} because that's very declarative etc. But that's not what fzf-lua does: https://github.com/ibhagwan/fzf-lua?tab=readme-ov-file#highlights it defines its own collection of highlight groups. I'm starting to lean towards doing what fzf-lua does to stay consistent (and linking fzf-org's highlight groups to fzf-lua's highlight groups by default). |
I think creating distinct highlight groups and linking them to either FzfLua or orgmode's highlight groups would be the best way forward. It also makes it much easier for a color scheme to add support for this plugin (I maintain one, and tend to add support for all the plugins I use, so perhaps I'm a bit biased on this, haha). Tbh, I think that adding a config option to change the highlight groups unnecessarily add to the startup time. Users that want to change the highlights can easily do so via Neovim's API, as long as you list the names of the highlight groups in the readme. If you take this approach, make sure not to overwrite the highlight group if it's already set by the user. Since I have experience working with HL groups I'd be happy to try implementing this if you want. Just let me know. |
Sure, that sounds reasonable. I'd greatly appreciate your contribution! (I personally hate fussing with highlights 🙃) |
No worries, I'm happy to contribute! Adding this to my agenda and I'll get started on it when I have a bit of time. |
The current highlighting system consists of some highlight groups I picked on a whim, and are not configurable, nor do they match the highlight groups used in nvim-orgmode.
Following discussion in #2, we can make the highlights of each headline level match how they look in nvim-orgmode. However, I’m personally not sure I want to make this the default because I’m not sure if I want a bunch of different colors in my fzf window.
The text was updated successfully, but these errors were encountered: