neanias / everforest-nvim

A Lua port of the Everforest colour scheme
233 stars 15 forks source link

Change color for symbols #15

Closed deniciocode closed 9 months ago

deniciocode commented 9 months ago

This small change will use Agua for symbols. Coding in ruby and having a lot of symbols makes the look of the color pink unpleasant IMO. Here is the look and feel in a screen short. Left are symbols in Pink and on the right are the symbols in Agua

image
neanias commented 9 months ago

Hi there! I'm also a Rubyist and prefer the pink that it is now, personally. If you'd like to have the highlight be changed on your personal copy, you can use the configuration settings:

require("everforest").setup({
  on_highlights = function (hl, palette)
    hl.TSSymbol = { link = "Aqua" }
  end
})
deniciocode commented 9 months ago

Hey @neanias, I understand your personal preference. IMO the default color for symbols should be used. From the original Repo it is Agua.

neanias commented 9 months ago

Not wishing to be overly pedantic, but in the original, TSSymbol is linked to Fg: https://github.com/sainnhe/everforest/blob/72f101bd63228a1a45012325e39c5280ec22c828/colors/everforest.vim#L533.

Either way, the point of my personal preference being for pink as opposed to keeping it in line with the original repo is vaild and I'm happy to set it back to match the original repo and add my own stylings locally.

deniciocode commented 9 months ago

@neanias I am fine with pendantic

I understand the point and thanks for highlighting the line of code out. The original repo does not really work with tree sitter, this is the reason I found this repo here to have full color support. When ruby is not installed for tree-sitter the the symbols are in Agua. When ruby is installed on tree-sitter the colors for the symbols are gone.

I am not sure if we should change to non colored symbols. It looks more like a lack of tree-sitter support. Here is an example for the color for non tree-sitter https://github.com/sainnhe/everforest/blob/72f101bd63228a1a45012325e39c5280ec22c828/colors/everforest.vim#L2522

neanias commented 9 months ago

Are you opening a new PR? Happy to accept your changes

deniciocode commented 9 months ago

@neanias I messed something up and needed to readd the changes. Sorry for the confusion. All conflicts are gone and it is ready to be merged

deniciocode commented 9 months ago

@neanias I am not able to merge these changes

image
neanias commented 9 months ago

Many thanks for adding that!