-
-
Notifications
You must be signed in to change notification settings - Fork 619
Apply opts.renderer.icons.padding to arrow icons #3113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Thanks for the really useful "transparent diagnostics", the problem is really clear. I can see it in my terminal when I enable arrows. Yes, let's fix this please. I'd be most grateful for a Pull Request - it looks like you have identified the change necessary. We will need to define another option, so as not to interrupt This functionality is historically fragile, so we will need to test all permutations and combinations of |
Once this is all tested and ready to merge, we can refactor the option in a backwards compatible manner. This is messy, so we do it last so as to avoid noise in the PR. renderer = {
icons = {
padding = {
icon = " ",
folder_arrow = "",
},
}, |
I used your naming idea for the new config attribute. I have done minimal testing but will try to test everything tomorrow. |
* Update padding.lua * add folder_arrow_padding * update help docs * refactor: renderer.icons.padding renderer.icons.padding -> renderer.icons.padding.icon renderer.icons.folder_arrow_padding -> renderer.icons.padding.folder_arrow * refactor: renderer.icons.padding --------- Co-authored-by: Alexander Courtis <alex@courtis.org>
When using devicons where nerd font is not "mono", the glyphs overhang the next character. Because of this an extra space of padding is added. When using nvim-tree in a floating window with winblend(transparency), the glyphs will overhang the character from the underlying window.
Describe the solution you'd like
add opts.renderer.icons.padding to the padding.get_arrows function
Additional context
using default opts.renderer.icons.padding
using U+00A0 (NO-BREAK SPACE) as opts.renderer.icons.padding
The text was updated successfully, but these errors were encountered: