Labeling the new "Add New Jobs (fuzzy)" interface as fuzzy is misleading
Categories
(Tree Management :: Treeherder, task)
Tracking
(Not tracked)
People
(Reporter: ahal, Unassigned)
Details
This is a minor thing, but I think it's worth fixing. There are two problems with calling this interface fuzzy:
-
It isn't fuzzy, it only handles substring matches. I think substring is fine here, but people might expect to be able to use fuzzy terms based on the name and then get confused when it doesn't work.
-
It makes people think the syntax is the same as |mach try fuzzy|. I.e, they might expect to be able to use fzf's
!,|,$, etc. operators and then be confused when those don't work.
My recommendation would be to change the name to Add new jobs by label.
| Reporter | ||
Comment 1•7 years ago
|
||
Or maybe even rename the original one too, so we could have:
Add jobs by symbol
Add jobs by label
| Reporter | ||
Updated•7 years ago
|
I've added renaming it to my pull request in bug 1546463. I think "Add New Jobs (Search)" better conveys what it can do, and it also (with that pull request) is able to search by either the label or the job symbol (but not both, due to a bug in the fuzzy search library we're using: https://github.com/krisk/Fuse/issues/302 ).
I've also filed https://github.com/krisk/Fuse/issues/306 about hopefully getting fzf's search syntax operators to work with fuse.js. Don't know if anything will come of that, but worth a shot.
Description
•