Results 268 comments of luukvbaal

> commit message "builtin ext_cmdline completion popupmenu" could be more specific. i e this adds the missing hook to support such a builtin popupmenu, but does not (yet) contain the...

Thanks. Might I suggest the following change to fix the pager highlights, lint errors and reduce the line count: ```diff diff --git a/runtime/lua/vim/_extui/messages.lua b/runtime/lua/vim/_extui/messages.lua index f684017cc1..a722c9a24d 100644 --- a/runtime/lua/vim/_extui/messages.lua +++...

Right, I don't see solution for extui other than just fetching these highlight definitions for each message (when they are used). A perhaps cleaner solution could be to add a...

Yes `priority` cannot solve this problem. It would look something like this: ```diff diff --git a/runtime/lua/vim/_extui/messages.lua b/runtime/lua/vim/_extui/messages.lua index 08b230737c..7761bfe45c 100644 --- a/runtime/lua/vim/_extui/messages.lua +++ b/runtime/lua/vim/_extui/messages.lua @@ -182,7 +182,9 @@ end --...

> so don't think it should drive a new nvim_buf_set_extmark feature. I agree, but I thought _if_ it makes sense as a general feature then why not. I pinged @bfredl...

Looks like we need to handle this case https://github.com/neovim/neovim/pull/28961#discussion_r1613208986, this repro hits the assert: https://github.com/neovim/neovim/blob/aa6b9c677d83d76d448c3bb0973bf8d14bfdf922/src/nvim/extmark.c#L120-L121

Thanks for the kind words and sorry for the late response. I'm not sure what could cause this error and I'm not able to reproduce this issue. Were you able...