813ethan

> > Patch 8.2.2427 and 8.2.2426 disallow changing windows in the completefunc. > > So with > > ``` > :wincmd P > setlocal ft=lspgfm > ``` > > make...

> Can you try to make sense of the solution [justmao945/vim-clang#140](https://github.com/justmao945/vim-clang/pull/140) ? it tried to move the problematic calls away from the autocomplete function? I'm not an expert about the...

my understanding is `LspSetPopupFileType()` is called to setup the completion popup's markdown highlighting, while it works for popup windows, it fails with preview windows bc of the error, while lazy...

> ok nvm `try ... catch` E565 here silenced the errors, but markdown isn't applied to the preview window > > https://github.com/yegappan/lsp/blob/5ded99487aab03513120a758fd5222d413284bf4/autoload/lsp/completion.vim#L631-L635 > > screen-20251105-181844.2.mp4 > the error still exists...

https://github.com/yegappan/lsp/blob/5ded99487aab03513120a758fd5222d413284bf4/autoload/lsp/completion.vim#L763-L766 actually can we even change the state of the preview window during `CompleteChanged`

> > Thank you, so doing it the roundabout `ShowCompletionDocumentation()` way > > ```viml > > # autoload/lsp/completion.vim (lines 453-460) > > :wincmd P > > :setlocal modifiable > >...

> > Changing windows while inside a completion function is forbidden now. > > I think changing means less ambiguously switching here; so we can still change, say the filetype,...

> > > Changing windows while inside a completion function is forbidden now. > > > > > > I think changing means less ambiguously switching here; so we can...

tried setting `completeopt+=preview` but it doesn't seem to work

alright thanks, are there any plans for an implementation?