nvim-bqf icon indicating copy to clipboard operation
nvim-bqf copied to clipboard

bqf slow

Open teto opened this issue 1 year ago • 3 comments

Feature description

I love bqf but it slows down moving between quicklist items noticeably (200ms - 500ms I suppose). I am using neovim master but this has been the case for months.

Describe the solution you'd like

I've seen https://github.com/kevinhwang91/nvim-bqf/issues/33, is it still up to date ?

At https://github.com/kevinhwang91/nvim-bqf#setup-and-description (Iw would suggest to start with "require'bqf'" else someone has to look up the name of the module to import), there is a "delay_syntax" option I am not sure would help. Does it delay highlighting so that bqf can display the file immediately ?

Additional context

I have treesitter disabled.

Now that I am writing this I wonder if I enable options that accentuate the slowness. Is there a way I can customize the bqf popup to disable (relative)number for instance ?

teto avatar Sep 05 '22 15:09 teto

I've seen https://github.com/kevinhwang91/nvim-bqf/issues/33, is it still up to date ?

Only add an extra winid for quickfix as a parameter, and the code should work.

there is a "delay_syntax" option I am not sure would help. Does it delay highlighting so that bqf can display the file immediately ?

Yes, unless your buffer content is large enough, shouldn't be slow.

I have treesitter disabled.

bqf only optimizes treesitter highlighting, vim regex highlighting may be slow.

Is there a way I can customize the bqf popup to disable (relative)number for instance ?

I'm not clear about what you said above, could you explain it clearly? Maybe a screenshot or video would help.

kevinhwang91 avatar Sep 05 '22 15:09 kevinhwang91

I'm not clear about what you said above, could you explain it clearly? Maybe a screenshot or video would help.

when bqf creates its preview window, I would like to be able to set window-local options such set nonumber, i.e., during an on_create_window callback or via an autocmd that can distinguish bqf preview windows from other windows.

I will try reenabling treesitter but the tradeoff for treesitter was negative for me (so far).

teto avatar Sep 05 '22 20:09 teto

when bqf creates its preview window, I would like to be able to set window-local options such set nonumber, i.e., during an on_create_window callback or via an autocmd that can distinguish bqf preview windows from other windows.

Don't support it for now. If you think it's necessary to customize the number, I can add an option like https://github.com/kevinhwang91/nvim-bqf/blob/aac1ff94a8b411a08810117f41e948743a4df69e/README.md#L203-L206.

I will try reenabling treesitter but the tradeoff for treesitter was negative for me (so far).

I have no any perf issue about treesitter after I customize the query.

  1. Remove some parser like comment, and reduce the injection language as little as possible, queries/foo/injections.scm.
  2. Reduce inefficient predicates, :h lua-treesitter-predicates for detail, queries/foo/highlights.scm

kevinhwang91 avatar Sep 05 '22 22:09 kevinhwang91