Auto re-highlight does not respect defined legacy highlight groups if new ones are default-linked.
Une pull request liée a déjà été fusionnée.
- #3360 par @xsbsy-hzbyp — fusionnée
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 70/100
Piste de recherche
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.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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
- Langage dominant
- Lua
- Étoiles
- 8.7k
- Forks
- 638
- Merge moyen
- 8 h 25 min
- PR mergées (30 j)
- 3
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de nvim-tree/nvim-tree.lua
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
nvim-tree/nvim-tree.lua#3362 · 1 commentaire ·
-
documentation
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
nvim-tree/nvim-tree.lua#3361 ·
-
Use vim.Option getters and setters instead of directly settingPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
nvim-tree/nvim-tree.lua#3357 · 1 commentaire ·
-
Migrate To vim.packOuvertedocumentation PR please
Difficulté 1/5 Moins d'une heure Accessibilité débutants 65/100
nvim-tree/nvim-tree.lua#3179 · 6 commentaires ·
-
upstream: neovim
Difficulté 4/5 3-5 jours Accessibilité débutants 38/100
nvim-tree/nvim-tree.lua#3344 · 1 commentaire ·
Toutes les issues de nvim-tree/nvim-tree.lua
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour
-
bug: splunk-hec-logging crashes with nil concatenation when error response json lacks text fieldPeut-être pris @bhuvan-somisetty l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 2 jours
-
enhancement good first issue help wanted
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
gabrielmongefranco/privatium#84 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
cataclysmbn/Cataclysm-BN#10516 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
silverbulletmd/silverbullet#2187 ·
Les mainteneurs répondent en général sous 2 jours