rcarriga / nvim-notify

A fancy, configurable, notification manager for NeoVim
MIT License
3.03k stars 81 forks source link

`wrapped-compact` render gives bad argument error #225

Closed Futarimiti closed 1 year ago

Futarimiti commented 1 year ago

Hi, I'm on macOS, using NVIM v0.9.2 on iTerm2.

Minimal reproducible configuration:

local notify = require 'notify'

vim.opt.termguicolors = true

notify.setup { render = 'wrapped-compact' }

vim.notify = notify

After sourcing this configuration, executing :lua vim.notify 'a' gives a bad argument error:

E5108: Error executing lua .../start/nvim-notify/lua/notify/render/wrapped-compact.lua:17: bad argument #1 to 'sub' (number expected, got nil)
stack traceback:
        [C]: in function 'sub'
        .../start/nvim-notify/lua/notify/render/wrapped-compact.lua:17: in function 'split_length'
        .../start/nvim-notify/lua/notify/render/wrapped-compact.lua:28: in function 'custom_wrap'
        .../start/nvim-notify/lua/notify/render/wrapped-compact.lua:49: in function 'render'
        ...ker/start/nvim-notify/lua/notify/service/buffer/init.lua:107: in function 'render'
        ...ack/packer/start/nvim-notify/lua/notify/service/init.lua:53: in function 'push'
        ...te/pack/packer/start/nvim-notify/lua/notify/instance.lua:77: in function 'notify'
        [string ":lua"]:1: in main chunk

All the other built-in renders have not caused any problems so far.

rcarriga commented 1 year ago

Should be fixed in latest commit :smile:

Futarimiti commented 1 year ago

Thank you! 👍