add lsp format operator mapping #546
Conversation
| def LspFormatFunc(type: string, visual_mode = v:false) | ||
| if visual_mode | ||
| exe "normal! gv:LspFormat\<cr>" | ||
| elseif type ==# 'line' | ||
| exe "normal! '[V']:LspFormat\<cr>" | ||
| elseif type ==# 'char' | ||
| exe "normal! `[v`]:LspFormat\<cr>" | ||
| endif | ||
| enddef |
There was a problem hiding this comment.
seems a bit mess, are you sure the logic was correct?
btw: the range format normally required lsp server gave it.
vs to local, not sure if it was ok, or if you made it really work, maybe ok too? 🤷
There was a problem hiding this comment.
Works for me and is a suggestion. Feel free to improve, I am stuck with Legacy Vim script for the moment in view of the divergent eco system. Likely the function has to go into autoload, if ever accepted.
There was a problem hiding this comment.
there is lsp#lsp#FormatExpr() already, though req lsp server gave the range format.
There was a problem hiding this comment.
In fact, I am not sure if the lsp server understands the range correctly yet; for example, formatting a single line of a comment with clangd formats the whole comment
As convenience complementing, say
xnoremap gw :LspFormat<cr>