Yagua / nebulous.nvim

Minimalist Collection of Colorschemes for Neovim Written in Lua
MIT License
184 stars 11 forks source link

Change LspReference colors to a more readable ones #8

Closed luissolanodev closed 3 years ago

luissolanodev commented 3 years ago

Hola amigo, noté que los grupos para resaltar las referencias del LSP (LspReference*) no eran legibles con los colores que tenían (letras naranja sobre fondo azul). Opté por aplicar solamente un fondo más claro.

A ver qué te parece bro, espero haberme explicado.

Yagua commented 3 years ago

Hola, muchas gracias por tus PR. Se ve interesante, sin embargo tu crees que podrias mostrarme una imagen como se ven reflejados estos cambios? (un tipo antes y despues) Gracias!

Estare atento a nuevas novedades.

luissolanodev commented 3 years ago

Claro que sí mi bro, sin ningún problema, de hecho ya lo había pensado pero se me pasó jaja, una disculpa.

Resaltado de LspReference actual: image

Resltado de LspReference con estos cambios: image

Revisar en la variable isLoading

Yagua commented 3 years ago

No te preocupes por nada, esta bien :). Por otro lado, esta genial lo que me muestras. Sin embargo hay una cosa mas que me gustaria saber, y es si este cambio se ve bien en todas las variantes disponibles en el colorscheme, puesto a que no es factible solo enforcarse unicamente en uno. Si Puedes observar si este camibio se ve bien en todos las variates, hacemos merge de una sin problema, porque reitero, es bueno el PR. Muchas gracias amigo.

luissolanodev commented 3 years ago

Twilight: image Se alcanza a distinguir un poco

Midnight: image Aquí de plano es el mismo color que el cursor cuando tienes la opcion set cursorline y por lo tanto no se nota nada.

Cual crees que sería una buena solución? cambiar los colores usados para el highlight? disculpa por mis PR tan pobres y poco probadas la verdad soy nuevo contribuyendo pero preferí hacerlo ya que no era tan complicado. Quedo atento a tus instrucciones bro.

Yagua commented 3 years ago

Now you can use the new "custom_colors" feature provided by the theme, so you can set your own set of colors, in case you want to change something in a custom way.

--Nebulous Settings
require("nebulous").setup {
  ....
  custom_colors = { -- this table can hold any group of colors with their respective values
    LineNr = { fg = "#5BBBDA", bg = "NONE", style = "NONE" },
    CursorLineNr = { fg = "#E1CD6C", bg = "NONE", style = "NONE" },

    -- it is possible to specify only the element to be changed
    TelescopePreviewBorder = { fg = "#A13413" },
    LspDiagnosticsDefaultError = { bg = "#E11313" },
    TSTagDelimiter = { style = "bold,italic" },
  }
}