Auto re-highlight does not respect defined legacy highlight groups if new ones are default-linked.
Ein zugehöriger Pull Request wurde bereits gemerged.
- #3360 von @xsbsy-hzbyp — gemerged
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 70/100
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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
- Vorherrschende Sprache
- Lua
- Sterne
- 8.7k
- Forks
- 642
- Ø Merge
- 2 T. 4 Std.
- Gemergte PRs (30 T.)
- 5
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus nvim-tree/nvim-tree.lua
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
nvim-tree/nvim-tree.lua#3362 · 1 Kommentar ·
-
documentation
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
nvim-tree/nvim-tree.lua#3361 ·
-
Use vim.Option getters and setters instead of directly settingEvtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
nvim-tree/nvim-tree.lua#3357 · 1 Kommentar ·
-
Migrate To vim.packOffendocumentation PR please
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 65/100
nvim-tree/nvim-tree.lua#3179 · 6 Kommentare ·
-
bug PR please reproduced
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 68/100
nvim-tree/nvim-tree.lua#3368 · 3 Kommentare ·
Alle Issues in nvim-tree/nvim-tree.lua
Ähnliche Issues
-
test-net.lua fails on linuxEvtl. vergeben @PatrickDahlin hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
Maintainer antworten meist innerhalb von 1 Tag
-
agent-review-finding
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 86/100
codymikol/multiverse.nvim#393 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
-
[alt:V] discontinuedOffen
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 68/100
pterodactyl/game-eggs#665 ·
Maintainer antworten meist innerhalb von 1 Tag