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

Do not show me a file if there are no results #8278

Closed
isidorn opened this issue Jun 28, 2016 · 6 comments
Closed

Do not show me a file if there are no results #8278

isidorn opened this issue Jun 28, 2016 · 6 comments
Assignees
Labels
feature-request Request for new features or functionality search Search widget and operation issues ux User experience issues
Milestone

Comments

@isidorn
Copy link
Contributor

isidorn commented Jun 28, 2016

#8184

Steps to Reproduce:

  1. Search / replace for a something
  2. Replace all inside a file
  3. Notice we still show that file with 'no matches'. I would expect that file is no longer in the list

screen shot 2016-06-28 at 11 36 40

@sandy081
Copy link
Member

@bpasero @isidorn

Following are the reasons, I want to retain the empty file

  • It is consistent with Search results - When a file entry is modified to not have search results
  • During replace, if user undo then file with results should be shown back. If we remove the entry then this will not happen.

@bpasero
Copy link
Member

bpasero commented Jun 28, 2016

Maybe change the wording if you are in replace mode so that it is more obvious?

@sandy081 sandy081 added the feature-request Request for new features or functionality label Jul 5, 2016
@sandy081 sandy081 added this to the July 2016 milestone Jul 5, 2016
@sandy081 sandy081 added the ux User experience issues label Jul 5, 2016
@sandy081
Copy link
Member

sandy081 commented Jul 6, 2016

I am thinking of removing this no matches in both Search and Replace and introducing following behavior

In Search results,

  • Editing a file so that it does not have any search results, will remove the file
  • Doing an undo will put it back in the results

In Replace results

  • Replacing all entries in a file (one by one or using replace all on a file) removes the file
  • Doing an undo will put it back in the results

Removing

  • File entry will not be added back if removed explicitly (Current behavior)

@bpasero
Copy link
Member

bpasero commented Jul 6, 2016

I like that 👍

@isidorn
Copy link
Contributor Author

isidorn commented Jul 6, 2016

Sounds good - would like to try it out before giving more feedback

sandy081 added a commit that referenced this issue Jul 7, 2016
sandy081 added a commit that referenced this issue Jul 7, 2016
@sandy081
Copy link
Member

sandy081 commented Jul 7, 2016

Pushed the changes, Feedback is welcome. Thanks.

@sandy081 sandy081 closed this as completed Jul 7, 2016
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality search Search widget and operation issues ux User experience issues
Projects
None yet
Development

No branches or pull requests

3 participants