Auto re-highlight does not respect defined legacy highlight groups if new ones are default-linked.
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 70/100
Línea de trabajo
Start in lua/nvim-tree/appearance/init.lua around the linked re-highlight logic at line 198, and read how highlight groups are checked and linked. Verify the behavior with a defined legacy group and a new group retaining only its default link. Done means re-highlighting respects the legacy definition in that case without changing existing behavior for explicitly defined new groups.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Description
If we have previously highlight default-ed a highlight group, then even after highlight clear, the current re-highlight logic would not respect legacy highlights even if they are defined, so for example, even if a colorscheme has a definition for NvimTreeGitNew, after our autocmd is run, NvimTreeGitNewIcon still links to PreProc.
Currently we only link to a legacy group if the new group is not defined and the legacy group is defined. Should we change it to "if the new group is not defined or is defined but has only the default link"?
Neovim version
NVIM v0.12.5
Operating system and version
Linux
Windows variant
No response
nvim-tree version
master
Clean room replication
none
Steps to reproduce
none
Expected behavior
No response
Actual behavior
No response
- Lenguaje dominante
- Lua
- Estrellas
- 8.7k
- Forks
- 638
- Merge medio
- 1 d 1 h
- PR fusionados (30 d)
- 3
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de nvim-tree/nvim-tree.lua
-
Use vim.Option getters and setters instead of directly settingPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
nvim-tree/nvim-tree.lua#3357 · 1 comentario ·
-
Migrate To vim.packAbiertodocumentation PR please
Dificultad 1/5 Menos de una hora Aptitud para principiantes 65/100
nvim-tree/nvim-tree.lua#3179 · 6 comentarios ·
-
upstream: neovim
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
nvim-tree/nvim-tree.lua#3344 · 1 comentario ·
-
Experimental Features: `session_restore_nvim`Quizá libre de nuevo @igorlfs la tomó hace 89 días y no hay ningún pull request abierto. Abierto
nvim-tree/nvim-tree.lua#3343 · 3 comentarios · 1 asignado ·
-
feature request
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
nvim-tree/nvim-tree.lua#3341 · 1 comentario ·
Todos los issues de nvim-tree/nvim-tree.lua
Issues similares
-
bug: splunk-hec-logging crashes with nil concatenation when error response json lacks text fieldPosiblemente ocupada @bhuvan-somisetty la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 2 días
-
enhancement good first issue help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
gabrielmongefranco/privatium#84 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
cataclysmbn/Cataclysm-BN#10516 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
silverbulletmd/silverbullet#2187 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100