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

Duplicate "Clear Search" and "No Settings Found" in settings-editor #86490

Closed
smilegodly opened this issue Dec 6, 2019 · 7 comments · Fixed by #86494
Closed

Duplicate "Clear Search" and "No Settings Found" in settings-editor #86490

smilegodly opened this issue Dec 6, 2019 · 7 comments · Fixed by #86494
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug search Search widget and operation issues verified Verification succeeded
Milestone

Comments

@smilegodly
Copy link
Contributor

smilegodly commented Dec 6, 2019

I noticed in the settings-editor that there are now duplicate clear buttons when the user searches but finds no result.

I propose getting rid of the "Clear Search" text in the bottom left corner as there is now a button in the search bar that achieves the same thing but with less clutter (in my opinion). This change would make the settings-editor more consistent with the keyboard shortcuts tab as well.

Capture

@vscodebot
Copy link

vscodebot bot commented Dec 6, 2019

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

@vscodebot vscodebot bot added the search Search widget and operation issues label Dec 6, 2019
@smilegodly smilegodly changed the title Duplicate "Clear Search" buttons Duplicate "Clear Search" buttons in settings-editor Dec 6, 2019
@roblourens roblourens added debt Code quality issues bug Issue identified by VS Code Team member as probable bug and removed debt Code quality issues labels Dec 6, 2019
@roblourens roblourens added this to the December 2019 milestone Dec 6, 2019
@roblourens
Copy link
Member

Yeah we can probably just get rid of the text link

@jzyrobert
Copy link
Contributor

What about the "Clear Filters" link?
Should there be an extra Clear Filters button that appears in the search bar itself, or let the link stay?

@smilegodly
Copy link
Contributor Author

smilegodly commented Dec 6, 2019

@jzyrobert Would you mind sending me a picture of the "Clear Filters" link ? Where exactly is the link ?
I will look into this.

@jzyrobert
Copy link
Contributor

@smilegodly
Code_2019-12-06_21-02-38

When you use one of the @ filters it appears.

@smilegodly
Copy link
Contributor Author

smilegodly commented Dec 6, 2019

For now I think we can let the 'Clear Filters' link stay. This issue was originally meant to get rid of the duplicates.
In another PR I could add a button to the search bar that adds this , and then we could get rid of that link though, although other tabs don't have that sort of button .
What do you think ? @roblourens

@smilegodly smilegodly changed the title Duplicate "Clear Search" buttons in settings-editor Duplicate "Clear Search" and "No Settings Found" in settings-editor Dec 6, 2019
@roblourens
Copy link
Member

Yeah the "Clear Filters" link is all about discoverability and I'd rather leave it

@vscodebot vscodebot bot locked and limited conversation to collaborators Jan 22, 2020
@mjbvz mjbvz added the verified Verification succeeded label Jan 29, 2020
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 search Search widget and operation issues verified Verification succeeded
Projects
None yet
4 participants