Auto re-highlight does not respect defined legacy highlight groups if new ones are default-linked.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 70/100
Hướng nghiên cứu
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.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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
- Ngôn ngữ chính
- Lua
- Star
- 8.7k
- Fork
- 638
- Merge trung bình
- 1 ngày 1 giờ
- Pull request đã merge (30 ngày)
- 3
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của nvim-tree/nvim-tree.lua
-
Use vim.Option getters and setters instead of directly settingCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
nvim-tree/nvim-tree.lua#3357 · 1 bình luận ·
-
Migrate To vim.packĐang mởdocumentation PR please
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 65/100
nvim-tree/nvim-tree.lua#3179 · 6 bình luận ·
-
upstream: neovim
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 38/100
nvim-tree/nvim-tree.lua#3344 · 1 bình luận ·
-
Experimental Features: `session_restore_nvim`Có thể làm lại được @igorlfs đã nhận 88 ngày trước và không có pull request nào đang mở. Đang mở
nvim-tree/nvim-tree.lua#3343 · 3 bình luận · 1 người được giao ·
-
feature request
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
nvim-tree/nvim-tree.lua#3341 · 1 bình luận ·
Tất cả issue của nvim-tree/nvim-tree.lua
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
AstroNvim/astrocommunity#1802 ·
-
Bug era/hc Miscellaneous Task
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 64/100
-
engine priority: 3 - wishlist wontfix
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
panel-attack/panel-game#780 ·