Statusline virtcol display doesn't respect neovim `positive count` requirement

[wroyca]

Contributing guidelines

• [X] I have read CONTRIBUTING.md
• [X] I have read CODE_OF_CONDUCT.md

Module(s)

Statusline

Description

Hello o/.

Neovim requires positive count which is currently not respected by the virtcol display (perhaps by design?) in Mini.Statusline location section:

For the sake of this example, if you, e.g., type d0 to delete the empty line, Neovim will return an error, instead expecting d1.

One way to solve this is to set the minimum value to 1:

MiniStatusline.section_location = function(args)
  -- Use virtual column number to allow update when past last column
  if MiniStatusline.is_truncated(args.trunc_width) then return '%l│%2v' end

  -- Use `virtcol()` to correctly handle multi-byte characters
  return string.format('%%l|%%L│%%2v|%-2d', math.max(vim.fn.virtcol("$") - 1, 1))
end

[echasnovski]

Thanks for the issue!

This is indeed intentional and allows to differentiate truly empty line and a line with one character (possibly whitespace).

Closing as behavior is by design.

---

Neovim requires positive count which is currently not respected by the virtcol display (perhaps by design?) in Mini.Statusline location section:

I am not sure how total number of columns in the current line is related to count.

For the sake of this example, if you, e.g., type d0 to delete the empty line, Neovim will return an error, instead expecting d1.

I can not reproduce. The d0 executes d with 0 motion with no errors. The d1 requires a motion/textobject after typing it.

[wroyca]

I can not reproduce. The d0 executes d with 0 motion with no errors. The d1 requires a motion/textobject after typing it.

I should have specified that it must be run from the command line - My bad!

[echasnovski]

I should have specified that it must be run from the command line - My bad!

Ah, then it surely doesn't have anything to do with 0 being used as number of columns in current line. Because :d operates on lines and uses {count} as number of lines.