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

Add autocomplete styles #1425

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

Conversation

merlindiavova
Copy link

Add style properties specific to autocomplete.

style.completion_background = { common.color "#ede7db" }
style.completion_text = { common.color "#000000" }
style.completion_type = { common.color "#4a3a8a" }
style.completion_text_active = { common.color "#8a290f" }
style.completion_infobar_divider = { common.color "#a60000" }
style.completion_infobar_background = { common.color "#fbf7f0" }
style.completion_text_accent = { common.color "#000000" }

Might be useful for others.

Add style properties specific to autocomplete.

```lua
style.completion_background = { common.color "#ede7db" }
style.completion_text = { common.color "#000000" }
style.completion_type = { common.color "#4a3a8a" }
style.completion_text_active = { common.color "#8a290f" }
style.completion_infobar_divider = { common.color "#a60000" }
style.completion_infobar_background = { common.color "#fbf7f0" }
style.completion_text_accent = { common.color "#000000" }
```

Might be useful for others.
@adamharrison
Copy link
Member

Hey, sorry it took so long to review this.

So we had a bit discussion in PR committee meeting number 8 about this; and we came to the conclusion that while we do wanna be able to add more colors like this, especially for plugins. Currenlty the mechanism to do this is just generally to put things in config.plugins.autocomplete.completiong_background or something.

Doing it in style like this, and having fallbacks causes problems with themes that aren't the default theme, because you'll never fallback (i.e., you'll always use the default theme values for these), due to the way the style stuff is inherited.

We'll put our a PR to address this with a new system of some kind, sometime soon; we can adjust this PR at that point to use it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants