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

"find all references" previously highlighted "find all references" word remains highlighted #17346

Closed
gujiman opened this issue Dec 15, 2016 · 7 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Milestone

Comments

@gujiman
Copy link

gujiman commented Dec 15, 2016

#- VSCode Version: 1.8.0

  • OS Version: win 7

Steps to Reproduce:

  1. Open any type of file where by a keyword may be used in multiple files. I.e we have 2 Files and both have keywords used in both files:

File 1:

function a(){
return 1;
}
b();


File 2:

function b(){
return 1;
}
a();


  1. Now select a() from file 1 and right click and select "find all references" to open the "find all references" box.
  2. You have to toggle both files in the "find all references" box (do it a couple of times).
  3. Close the "Find in File" box.
  4. Now select b() and repeat step 2 and 3. As you will see that the previously highlighted b() is still highlighted.
@rebornix
Copy link
Member

search-highlight

@gujiman can you help confirm above gif represents what you are describing? Are you saying that in step 5, b() is still highlighted in the sidebar?

cc @roblourens it looks like that the search sidebar is not cleaned even though that it's hidden before we run Find in files

@rebornix rebornix added the info-needed Issue requires more information from poster label Dec 15, 2016
@gujiman gujiman changed the title "find in files" previously highlighted "find in files" word remains highlighted "find in references" previously highlighted "find in references" word remains highlighted Dec 18, 2016
@gujiman gujiman changed the title "find in references" previously highlighted "find in references" word remains highlighted "find all references" previously highlighted "find all references" word remains highlighted Dec 18, 2016
@gujiman
Copy link
Author

gujiman commented Dec 18, 2016

hi,

Im really sorry about this but the problem was in "find all references" feature not "find in files" feature!

@gujiman
Copy link
Author

gujiman commented Dec 19, 2016

@roblourens just wondering why the issue was closed. I have sent the following screenshot which shows the bug (which seems that the highlighted keyword from the the previous "find all references" result "a()" does not go away when doing a "find all references" for "b()"):

@rebornix rebornix reopened this Dec 19, 2016
@rebornix
Copy link
Member

@gujiman no worries. it' a valid issue about Find all reference.

@roblourens
Copy link
Member

Sorry for closing it, I misread

@gujiman
Copy link
Author

gujiman commented Dec 20, 2016

no probs. Brilliant software by the way

@rebornix rebornix added editor bug Issue identified by VS Code Team member as probable bug and removed info-needed Issue requires more information from poster labels Dec 22, 2016
@rebornix
Copy link
Member

@gujiman excellent finding actually, it's a bug in the reference widget. I like your reproduce steps.

@jrieken jrieken assigned jrieken and unassigned roblourens and alexdima Dec 28, 2016
@jrieken jrieken added this to the January 2017 milestone Dec 28, 2016
@jrieken jrieken closed this as completed Dec 28, 2016
@ramya-rao-a ramya-rao-a added the verified Verification succeeded label Jan 27, 2017
@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
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

6 participants