Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Use Fuzzy Search" button toggle - a weird UI pattern #156322

Closed
isidorn opened this issue Jul 26, 2022 · 16 comments · Fixed by #158616
Closed

"Use Fuzzy Search" button toggle - a weird UI pattern #156322

isidorn opened this issue Jul 26, 2022 · 16 comments · Fixed by #158616
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders polish Cleanup and polish issue terminal-shell-integration Shell integration, command decorations, etc. verified Verification succeeded
Milestone

Comments

@isidorn
Copy link
Contributor

isidorn commented Jul 26, 2022

Testing #156002

I find it weird that the "Use Fuzzy Search" is a big blue button. I think it would be styled more like the toggles in the editor find widget. Or it should be a checkbox.
The blue button is too strong, and I would expect that it executes something a closes the quick pick

fyi @misolori

Screenshot 2022-07-26 at 18 51 40

@meganrogge
Copy link
Contributor

Yes we agree and have had discussions about a better icon that aligns with other toggles

@meganrogge meganrogge added polish Cleanup and polish issue terminal-shell-integration Shell integration, command decorations, etc. labels Jul 26, 2022
@miguelsolorio
Copy link
Contributor

This is also being tracked in #156179 where we're looking at all of the inputs w/ actions/toggles

@Tyriar
Copy link
Member

Tyriar commented Aug 8, 2022

Depends on #156179

@Tyriar
Copy link
Member

Tyriar commented Aug 17, 2022

UX sync feedback:

Prefer a toggle over a dropdown as it's a search mode/how it's search, vs a dropdown which is typically what is searched/filtered.

It seems the consensus was some icon similar to this to mean "fuzzy search", we can also avoid mentioning "default"/"continuous"/"contiguous" if it's a toggle.

image

We may want the fuzzy search toggle to be remembered and/or in a setting, fzf users may want fuzzy search by default.

@meganrogge
Copy link
Contributor

Wouldn't it be better to display what it actually does like we do with the Aa case specific search toggle?

@Tyriar
Copy link
Member

Tyriar commented Aug 17, 2022

Icon request: microsoft/vscode-codicons#132

@meganrogge
Copy link
Contributor

By having it underline either a continuous segment or separate ones

@Tyriar
Copy link
Member

Tyriar commented Aug 17, 2022

@meganrogge that was the idea behind these ones:

image
image
image

The idea seemed solid but I think in practice they are pretty ugly

@meganrogge
Copy link
Contributor

I far prefer those because it tells me what's going on. The other makes me think of zoom

@Tyriar
Copy link
Member

Tyriar commented Aug 17, 2022

While the magnifying glass can mean zoom, it's only used for search in VS Code afaik

image

@meganrogge
Copy link
Contributor

Yeah but a button's icon should indicate what it does - the button doesn't search - it changes the type of filter used

@meganrogge
Copy link
Contributor

IMO, that's precisely why it's confusing - I'm already searching via the input box - not sure what a search with a squiggly line will do

@Tyriar
Copy link
Member

Tyriar commented Aug 18, 2022

@meganrogge fuzzy search is more of a search mode which are typically represented as toggled. Usually filters (the dropdown) refine the search space, so a filter-like icon isn't appropriate here imo.

the button doesn't search - it changes the type of filter used

It defines how a search happens, so the icon indicates search + a modifier which is a fuzzy line.

@misolori revisiting those ideas that @meganrogge prefers, the second one seems decent and not too busy, what do you think? Maybe better with a-c or a-d as b could indicate it comes after a? We could do a team poll

image

@Tyriar
Copy link
Member

Tyriar commented Aug 18, 2022

Some more iterations on that idea:

image

@miguelsolorio
Copy link
Contributor

I think those are very minor/small changes that users might confuse them with "Match Whole Word", and I think the feedback from the UX sync was those wouldn't work as best.

Are we trying to re-visit the icon itself again? I don't think we're going to find an icon that best represents it as it's a hard concept to encapsulate in an icon. I think we should try the one we just merged in and see what the feedback is.

dabreadman pushed a commit to THIS-IS-NOT-A-BACKUP/vscode that referenced this issue Aug 18, 2022
@VSCodeTriageBot VSCodeTriageBot added the unreleased Patch has not yet been released in VS Code Insiders label Aug 22, 2022
@Tyriar Tyriar added the bug Issue identified by VS Code Team member as probable bug label Aug 22, 2022
@Tyriar
Copy link
Member

Tyriar commented Aug 22, 2022

To verifier: Verify the toggle works as expected/like the button before it

@VSCodeTriageBot VSCodeTriageBot added insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Aug 23, 2022
@isidorn isidorn added the verified Verification succeeded label Aug 24, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Oct 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders polish Cleanup and polish issue terminal-shell-integration Shell integration, command decorations, etc. verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants