jdinhify / vscode-theme-gruvbox

Gruvbox theme for vscode based on https://github.com/morhetz/gruvbox
https://marketplace.visualstudio.com/items?itemName=jdinhlife.gruvbox
MIT License
274 stars 63 forks source link

Peekview keyword has no highlight group. #20

Closed serend1p1ty closed 4 years ago

serend1p1ty commented 5 years ago

image Now it's default ugly red.

jdinhify commented 5 years ago

Thanks for reporting this @ppnman 👌 I don't use peekView often though, so not sure what would be the best background, would you be able to suggest a colour?

serend1p1ty commented 5 years ago

image What do you think of the color(#ac6913) ? image Or this(#23395C)? Or you can reference other colortheme. BTW, your gruvbox colorscheme is great, I love it!

serend1p1ty commented 5 years ago

BTW, I suggest that the foreground of the item seleceted should be lighter than other entries. Like this: (Solarized Dark) image Now it is darker than other entries. (Gruvbox Medium) image

serend1p1ty commented 5 years ago

@jdinhify So the final effect is: image

// PEEK VIEW
    "peekView.border": "#3c3836",
    "peekViewEditor.background": "#3c383650",
    "peekViewEditorGutter.background": "#3c383650",
    "peekViewEditor.matchHighlightBackground": "#ac6913", #Modified
    "peekViewResult.background": "#3c383650",
    "peekViewResult.fileForeground": "#ebdbb2",
    "peekViewResult.matchHighlightBackground": "#8ec07c30",
    "peekViewResult.selectionBackground": "#8ec07c30",
    "peekViewResult.selectionForeground": "#FFFFFF", #Modified
    "peekViewTitle.background": "#3c383650",
    "peekViewTitleDescription.foreground": "#bdae93",
    "peekViewTitleLabel.foreground": "#ebdbb2",
jdinhify commented 5 years ago

thanks @ppnman, the color look cool, but I think it's better to stick with the colors in gruvbox palette, would you be able to give the colors.txt a try https://github.com/jdinhify/vscode-theme-gruvbox/blob/master/colors.txt

try

"peekViewEditor.matchHighlightBackground": "#fe801980",
"peekViewResult.selectionForeground": "#fbf1c7"

note that you can use 2 chars (00 -> ff) after the color code to specify the alpha, eg. #fe801980 is the orange 2 in colors.txt (fe8019) with 80 as alpha

serend1p1ty commented 5 years ago

@jdinhify Very good, I like it. image

jdinhify commented 5 years ago

Thanks @ppnman , would you be able to submit a PR for it? I'm happy to do it if you can't.

It'd be great if you can do for other variants, but no pressure, I can have a follow up commit to address other variants

serend1p1ty commented 5 years ago

@jdinhify I am really sorry that I just saw your reply. There are so many things going on there days so that I have no time to browse GitHub. Of course, I am glad to submit a PR for it. But maybe it takes one week to finish it because I need to consider what color is better in light colorscheme.

jdinhify commented 5 years ago

That's alright @ppnman , please take as much time as you need.

Thank you

serend1p1ty commented 4 years ago

I am sorry to close the issue because I don't use vscode anymore. BTW, both the plugin and author are great.