search

laloch / xontrib-fzf-widgets

Set of fzf widgets for xonsh
GNU General Public License v3.0
33 stars 18 forks source link

Remove --no-sort to allow sorting results #6

Closed CouleeApps closed 4 years ago

CouleeApps commented 4 years ago

Not sure why you pass --no-sort but it makes the results way worse. This just removes that so the results are sorted based on what you type.

laloch commented 4 years ago

Not sure why you pass --no-sort but it makes the results way worse.

It's because it's a history - I want the results to be sorted chronologically. What about rather adding --bind=ctrl-r:toggle-sort option?

laloch commented 4 years ago

OK, you might be right. Other shells do sort by default.

laloch commented 4 years ago

Thanks @CouleeApps!

CouleeApps commented 4 years ago

For reference, the sorting is still done by time, it's just also sorted when you search.

CouleeApps commented 4 years ago

Or rather, when you search it sorts by matches first, time second (which is what i assume you want)