Command completion with tab key

[MaxGyver83]

In the command prompt, press <tab> to get a list of all available commands and pick one (using vis-menu). This works also after typing the first letters of a command (p.e. :la<tab>).

Implements #1167 (and #100 more or less).

I'm aware that @martanne wants to avoid this feature. But I want to share it anyway for those who want to apply this patch to their local vis build.

[mcepl]

See also https://lists.sr.ht/~martanne/devel/%3C20240324225636.9318-1-mcepl@cepl.eu%3E for a discussion how to remove a warning caused by this patch.

[mcepl]

I have a deep suspicion that 4970df0 breaks this patch.

[fischerling]

I have a deep suspicion that 4970df0 breaks this patch.

It does indeed. But it is easily resolveable (a742dc3).

[MaxGyver83]

It does indeed. But it is easily resolveable (a742dc3).

Cool, thank you!

[fischerling]

It does indeed. But it is easily resolveable (a742dc3).

Cool, thank you!

You are welcome. Please double check it. But I use it on my main branch for some time now.

[MaxGyver83]

You are welcome. Please double check it. But I use it on my main branch for some time now.

I have rebased and force-pushed it. It seems to work.

[rnpnr]

I'm honestly on the fence about this. I know martanne wants to avoid this but the code is trivial and it aids in discoverability which in my opinion is one of the biggest marks of a good piece of software (I don't mean via tutorials/help pages but those are nice too).

Of course there are more efficient ways of doing this, especially if vis stored the available commands in a trie, but with the code that is already here this is pretty good.

[mcepl]

It does indeed. But it is easily resolveable (a742dc3).

This didn’t work for me (:t<TAB> does nothing), but I will try the latest commit yet.

[machinedgod]

Just as a side-observer, would this get merged or it will forever remain an optional patch (because Marc didn't think its the right way forward 10y ago - not being sarcastic, 10y is a long time for software evolution and change of opinion)

[mcepl]

@MaxGyver83 FYI, https://lists.sr.ht/~martanne/devel/patches/60055

And yes, it is becoming ridiculous that this PR hasn’t been merged yet.