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

Gruvbox theme has disorienting palette for selected regions #4625

Closed
kronodeus opened this issue Nov 7, 2022 · 1 comment
Closed

Gruvbox theme has disorienting palette for selected regions #4625

kronodeus opened this issue Nov 7, 2022 · 1 comment
Labels
C-enhancement Category: Improvements

Comments

@kronodeus
Copy link
Contributor

kronodeus commented Nov 7, 2022

Screen Shot 2022-11-06 at 8 37 22 PM

Top part of the screenshot above is the OOTB Gruvbox theme. As you can see, it's quite offensive to the eyes and makes it hard to see exactly where the selected region begins and ends at a glance. The bottom part of the screenshot is the theme after a couple tweaks to the palette.

Not sure if anyone actually prefers the way it is now, but it seems untenable to me.

Here's the modified settings from the screenshot:

"ui.selection" = { bg = "bg2" }
"ui.cursor.primary" = { bg = "fg4", fg = "bg1" }
"ui.cursor.match" = { bg = "bg3" }

Opened a pull request for the same here: #4626

@the-mikedavis
Copy link
Member

Closed in #4626

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: Improvements
Projects
None yet
Development

No branches or pull requests

2 participants