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

Default :terminal palette colors could have better contrast in GUIs #28916

Open
craigmac opened this issue May 22, 2024 · 7 comments
Open

Default :terminal palette colors could have better contrast in GUIs #28916

craigmac opened this issue May 22, 2024 · 7 comments
Labels
defaults issues or PRs involving changing the defaults display redraw, layout, presentation enhancement feature request highlight terminal built-in :terminal or :shell
Milestone

Comments

@craigmac
Copy link
Contributor

craigmac commented May 22, 2024

Originally posted by @craigmac in #26857 (comment)

The current :terminal palette colors when using neovim from a gui results in poor contrast, in particular the blue on black combination is difficult to read.

Can we improve the nvim --clean experience for GUI users, like in neovide, and only when has('gui_running') (or however we detect that a GUI is attached/shown). My preference would be to use the default colorscheme shipped with 0.10 as the palette, as it provides a unified look and it has a better readability. Barring that, something like Chris Kempson's 'default dark' (recent Alacritty's switched to this from Tomorrow Night) is battle-tested and works better than the default xterm palette.

@craigmac
Copy link
Contributor Author

craigmac commented May 22, 2024

Some images from Neovide 0.13 running Nvim 0.10 branch on macOS latest:

Example at the bottom here of blue fg on black bg issue:

Screenshot 2024-05-22 at 11 05 00 AM

Blue on black issue here much more obvious, also white on green combo is poor:

Screenshot 2024-05-22 at 11 05 56 AM

@justinmk justinmk added enhancement feature request defaults issues or PRs involving changing the defaults terminal built-in :terminal or :shell display redraw, layout, presentation highlight labels May 23, 2024
@justinmk justinmk added this to the backlog milestone May 23, 2024
@justinmk
Copy link
Member

Isn't that controlled by $LSCOLORS or some other shell-related env var?

@clason
Copy link
Member

clason commented May 23, 2024

I agree that my "terminal colors should match parent shell" objection does not apply here, I am still hesitant to use the Neovim colors for terminal colors, for the following reason: Unlike Vim highlight groups, these colors (at least the first eight) are named by the ANSI standard (black, red, green, yellow, blue, magenta, cyan, white). And since the official Neovim branding is so minimal, we can't match these expectations.

@justinmk
Copy link
Member

And since the official Neovim branding is so minimal, we can't match these expectations.

Nonsense. There is no requirement that the colors match the names, only that they are distinguishable.

@clason
Copy link
Member

clason commented May 23, 2024

@justinmk
Copy link
Member

What part are you quoting? I don't see any requirement on those pages about the colors needing to match the names. Otherwise they would not be configurable.

@clason
Copy link
Member

clason commented May 23, 2024

The named list of colors. I never said anything about hard requirements, only about expectations. (And I already regret saying anything at all.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
defaults issues or PRs involving changing the defaults display redraw, layout, presentation enhancement feature request highlight terminal built-in :terminal or :shell
Projects
None yet
Development

No branches or pull requests

3 participants