[REQ] Display buffer name as a header when grepping through project?

[Icy-Thought]

I don't know how to go on about implementing this, and because of this I am here asking for help.

When grepping a new project with many files located inside of it and navigating quickly, I find that pinning the buffer name for the duration of that grep being a useful feature that consult-ripgrep could add. Instead of treating it as a normal line, maybe having it placed as a header in that consult-ripgrep buffer would be a better idea?

Thoughts?

[Icy-Thought]

I just noticed that the buffer name is being pinned when scrolling from bottom to top. Wonder why that is not the case when going from top to bottom?

[minad]

I don't understand this feature request. Pinning the buffer name vs. treating it as a normal line? Which buffer? Can you please explain this a bit better or create a mock-up screenshot?

[Icy-Thought]

Apologies for the confusion.

New buffer, same search:

Moving a little down the line of that file:

Instead of retaining the line below, it becomes hidden throughout the search. Keeping it present is IMO very useful!

[Icy-Thought]

Does that make sense?

[minad]

I see. This is intentional to not waste too much title space. In Vertico there is this code:

    ;; Drop excess lines
    (setq lines (nreverse lines))
    (cl-loop for count from (length lines) above vertico-count do
             (if (< curr-line (/ count 2))
                 (nbutlast lines)
               (setq curr-line (1- curr-line) lines (cdr lines))))

In order for the title to always stick you could write:

    ;; Drop excess lines
    (setq lines (nreverse lines))
    (cl-loop for count from (length lines) above vertico-count do
                 (nbutlast lines))

Maybe I could improve the criterion slightly. I will experiment.

[Icy-Thought]

That code block did work as intended! Now here comes the question, is there a way to have that behavior become the default for vertico without having to maintain a local version of vertico.el in my config? I am very new to this form of customization, and because of this i am somewhat lost.

[minad]

That code block did work as intended!

No, it does not work under all circumstances. So we cannot make this the default or add it as a configuration option. I think this whole vertico--arrange-candidates logic must be rewritten to behave better when group titles are present. But this is more work and more difficult. You can give it a stab if you are interested.