search

rcarriga / nvim-notify

A fancy, configurable, notification manager for NeoVim
MIT License
2.88k stars 72 forks source link

[bug] `attempt to index a number value` in multiple places after recent fixes in neovim #252

Closed embe221ed closed 4 months ago

embe221ed commented 5 months ago

Hello, I've noticed that building NeoVim with this commit breaks the plugin. The floating windows are displayed incorrectly, without text, like this:

Screenshot 2024-02-09 at 10 38 20

I believe, the fix would consist of removing the [false] parts, I guess they were used as a workaround for something.

galk-research commented 5 months ago

Same problem happening after latest update through AstroNvim. Sorry, can you elaborate on the fix?
In the meantime, manually installed neovim 0.95. The nightly build PPA breaks the plugin. The stable PPA has neovim versions below 0.8.

utrumo commented 5 months ago

On arch now i just rebuild neovim to previous commit from src:

paru --chroot -Rsun neovim-nightly-bin tree-sitter paru --chroot --fm vim -S neovim-git add commit hash in neovim-git.PKGBUILD:

  "git+https://github.com/neovim/$_pkgname.git"
to
  "git+https://github.com/neovim/$_pkgname.git#commit=02cc84b4d9d8efdd27c3f60fef11e02297051fab"
pfernandom commented 4 months ago

This error is getting a bit annoying and I'm to lazy to move back to pre-nightly nvim, so I posted a PR to fix this issue.

tan-wei commented 4 months ago

Met the same issue when update neovim-nightly.

loqusion commented 4 months ago

Can confirm #253 fixes the issue for me