blink.cmp icon indicating copy to clipboard operation
blink.cmp copied to clipboard

Separate `winopts` for `cmdline` completions.

Open voidstar-null opened this issue 11 months ago • 4 comments

Feature Description

Since cmdline completions are completely different from normal completion. There should be separate winopts for them.

What I desire is something like this:

blink.cmp.config({
...
menu = {
cmdline = {
position = {...},
winopts = {...}, options like , max number of entries to display, and other stuff 
}
...
} 

voidstar-null avatar Dec 12 '24 19:12 voidstar-null

This would make many more configuration options dynamic which could increase complexity quite a bit. Could you explain your use case for this?

saghen avatar Dec 13 '24 03:12 saghen

Okay, so assume I have set no. of entries for normal completion menu to be 8 cuz I dont want that menu to hog the main window as I'm coding something. But since the winopts are global for every completion menu, the cmdline completion menu will also have max. 8 items , thus it will display less entries for cmdline completion too which I dont think anyone would want that (I hope you understand what I'm trying to say).

Also I dont want the kind icons or kind name to be displayed for cmdline completion menu, it just widens the menu for nothing ( as kind for every cmdline completions is the same)

This would make many more configuration options dynamic which could increase complexity quite a bit.

Totally agree with this. Maybe make the cmdline completion menu inherit options from global menu options and add a way so we can override options for cmdline menu?

voidstar-null avatar Dec 13 '24 07:12 voidstar-null

Also, implementing something like this would also solve cases like #523 .

voidstar-null avatar Dec 13 '24 07:12 voidstar-null

+1 This would be really nice

isak102 avatar Dec 14 '24 17:12 isak102

You may now change the rendering but not the winopts. I think I'll keep this as is for v1, but I'll revisit this for v2

saghen avatar Feb 19 '25 18:02 saghen