rakr / vim-one

Adaptation of one-light and one-dark colorschemes for Vim
MIT License
1.98k stars 196 forks source link

Define colors for the integrated terminal #120

Open rvighne opened 3 years ago

rvighne commented 3 years ago

This sets the 16 default colors within the Vim 8 integrated terminal (via g:terminal_ansi_colors) so that terminal apps look as expected. The colors are in terms of the already-defined variables, so it should stay consistent and work in light mode too.

Here's a comparison with One Half Dark from Windows Terminal. Note that WT decided to use a very bright color for white, but I've chosen to stick with the colorscheme so it looks better next to actual editor buffers.

Comparison with Windows Terminal's "One Half Dark"

lygamac commented 3 years ago

Terminal color was removed on #104 because the colors were inconsistent when using a light theme.

I second to re-add the colors, provided that we add support to the light theme too.

rvighne commented 3 years ago

These colors use the variables defined by the theme, like syntax_bg/syntax_fg. Since those are set conditionally based on the light/dark mode, I believe it works fine in that case. Here is a screenshot:

Screenshot of Vim terminal colors in light mode

Other terminal apps also look OK (actually I'm not at all a fan but actual light mode users can judge whether this looks correct 😁):

image

rvighne commented 2 years ago

Due to inactivity from the maintainers, I've merged this (and some other long-standing PR's) into my fork: https://github.com/rvighne/vim-one/pull/3