feat(#3113): add renderer.icons.folder_arrow_padding

[smchunn]

resolves #3113

default: opts.renderer.icons.folder_arrow_padding = " "

[alex-courtis]

You legend @smchunn

I'll get to a review and test over the weekend.

[alex-courtis]

Looks like you'll need to update help: https://github.com/nvim-tree/nvim-tree.lua/blob/master/CONTRIBUTING.md#documentation

[alex-courtis]

Working nicely.

• add new option table and legacy fallback

Full test matrix for config items that affect padding:

renderer.indent_width

renderer.icons.padding
renderer.icons.folder_arrow_padding

renderer.icons.show.folder_arrow

renderer.indent_markers.enable
renderer.indent_markers.inline_arrows
renderer.indent_markers.icons ...

That's not practical. Testing just those that affect get_arrows.
Directory and not directory.
Uncomment each test as you go

renderer.icons.glyphs.folder.arrow_closed = ">"
renderer.icons.glyphs.folder.arrow_open = "|"
renderer.icons.glyphs.folder.default = "c"
renderer.icons.glyphs.folder.open = "o"

-- renderer.icons.show.folder_arrow = false

-- renderer.icons.show.folder_arrow = true
-- renderer.indent_markers.enable = false
-- renderer.icons.folder_arrow_padding = "-"

-- renderer.icons.show.folder_arrow = true
-- renderer.indent_markers.enable = false
-- renderer.icons.folder_arrow_padding = ""

-- renderer.icons.show.folder_arrow = true
-- renderer.indent_markers.enable = true
-- renderer.icons.folder_arrow_padding = "-"

-- renderer.icons.show.folder_arrow = true
-- renderer.indent_markers.enable = false
-- renderer.icons.folder_arrow_padding = "----"

-- renderer.icons.show.folder_arrow = true
-- renderer.indent_markers.enable = true
-- renderer.icons.folder_arrow_padding = "----"

-- renderer.icons.glyphs.folder.arrow_closed = ""
-- renderer.icons.show.folder_arrow = true
-- renderer.indent_markers.enable = true
-- renderer.icons.folder_arrow_padding = "-"

-- renderer.icons.glyphs.folder.arrow_open = ""
-- renderer.icons.show.folder_arrow = true
-- renderer.indent_markers.enable = true
-- renderer.icons.folder_arrow_padding = "-"

[alex-courtis]

This is looking really good.

Let's refactor the options now:

  renderer = {
    icons = {
      padding = {
        icon = " ",
        folder_arrow =  " ",
      },
  },

We do this via legacy.lua and document it in :help nvim-tree-legacy-opts.

renderer.icons.webdev_colors looks like a good one to emulate - it moves a higher level option into a new subtable.

[alex-courtis]

Apologies, I steered you down a very incorrect path with the legacy conversion. Using the util is not possible as the source and destination are the same.

In the interests of speed I've pushed up a fix to your branch, please tell me if you don't prefer that.

Test cases passed:

empty config

renderer.icons.padding = nil

renderer.icons.padding = {}

renderer.icons.padding = "i"

• fails validation
  renderer.icons.padding = 99

renderer.icons.padding.icon = "i"
renderer.icons.padding.folder_arrow = "a"

• fails validation
  renderer.icons.padding.icon = 99
  renderer.icons.padding.folder_arrow = "a"

• fails validation
  renderer.icons.padding.icon = "i"
  renderer.icons.padding.folder_arrow = "a"

[alex-courtis]

• Please re-test following my changes