Unique path_prefix

[phy1729]

Possibly add a new style path_prefix_unique when the path prefix only matches a single file for a hint to hit tab.

[danielshahaf]

Reminds of of #148, which was a similar idea for command words.

[phy1729]

Should path_prefix_unique_pathseparator fall back to path_prefix_unique or path_prefix_pathseparator or something else? I'm inclined towards the former, but not strongly.

[danielshahaf]

I'd have voted for the latter, since that would maintain the distinction between a terminal separator and the characters before it, and those characters would be highlighted as path_prefix_unique anyway which would serve to convey the uniqueness...

...but I don't highlight path separators differently to other characters in pathnames. Let's ask the authors of the pathseparators PR (#136 #260).

@blaenk @m0vie WDYT regarding @phy1729's question?

[blaenk]

I'd have voted for the latter, since that would maintain the distinction between a terminal separator and the characters before it, and those characters would be highlighted as path_prefix_unique anyway which would serve to convey the uniqueness...

Thanks for the ping. IIUC, I do agree with this. It'd feel weird to have the path separators suddenly change/remove highlighting when the prefix became unique, reminiscent of the flickering that (used to?) happen.

[phy1729]

those characters would be highlighted as path_prefix_unique anyway which would serve to convey the uniqueness...

Right, indeed the latter makes more sense. I'll open a PR after #512 since they'll conflict.

[m0vie]

I'd have voted for the latter, since that would maintain the distinction between a terminal separator and the characters before it, and those characters would be highlighted as path_prefix_unique anyway which would serve to convey the uniqueness...

I also agree with this.

[Sarke]

I think this is a great suggestion. How can this be implemented?

[danielshahaf]

There's a (( $#tmp > 0 )) && REPLY=path_prefix on line 978 in current HEAD (I can't permalink, sorry). Check if ! (( $+tmp[2] )). We may be able to optimize this further with glob qualifiers: for example, integer foo; f() { (( ++foo )) }; : *(N+f).

(edit: The optimization example probably needs to do reply=(), in order to avoid constructing a temporary array.)

[danielshahaf]

There's also *(Y2) but it's not available in all supported zsh versions.

Finally, I wonder if perchance there already exists a zsh plugin that provides this functionality.