prabirshrestha / vim-lsp

async language server protocol plugin for vim and neovim
MIT License
3.07k stars 303 forks source link

Fix multiple diagnostic virtual text per-line of same severity in vim #1537

Open dylanahsmith opened 5 months ago

dylanahsmith commented 5 months ago

Remove the prop_remove just before prop_add in place_virtual_text, since this function is called after clear_virtual_text is called to clear the previously added virtual text.

That way, if there are multiple virtual text to place on the same line with the same severity, the following ones don't replace the former ones. Besides, the branch for neovim wasn't removing any property, so this should make it consistent with neovim.