multi select for history (ctrl+r) puts timestamps in front of all but the first command

[acegallagher]

Before proceeding...

• [ X] My CLIs meet the minimum version requirements
• [ X] I have read through the troubleshooting guide
• [ X] If I updated fzf.fish for the first time in over 6 months, I read the migration guides to make sure I'm caught up

Describe the bug

ctrl+r and then tab to select multiple entries puts the timestamp in front of the entries, like:

I imagine this is repeatable behavior for the package, but I'm happy to help debug of course if it's somehow only my setup. I'm also looking at the code now and trying to see if I could submit a pull request.

Environment

Versions installed: bat 0.21.0 (405e5f74) fd 8.4.0 fzf 0.31.0 (a0ef898) fish, version 3.4.1

• terminal: konsole
• OS: Linux 5.18.14

Thanks much for putting together something so well thought out.

[acegallagher]

Ah yeah, I have a pull request, just needed to change a few lines. Here is the code snippet:

set -U commands_selected # empty list to loop over selected commands, in case multiple
for c in (string split \n $command_with_ts)
    set --append commands_selected (string split --max 1 " │ " $c)[2]
end
commandline --replace -- $commands_selected

Incoming!

[PatrickF1]

Hi @acegallagher, the search history feature does not multi-select by default, so you must've configured it to be multi-selectable. What's your use case for selecting multiple commands?