-
Notifications
You must be signed in to change notification settings - Fork 30.3k
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
Allow to filter list by reference kind #64528
Comments
/duplicate of #50062 |
Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for existing issues here. See also our issue reporting guidelines. Happy Coding! |
@jrieken I'm not sure that this is duplicate of mentioned issue. In particular references inside of import statements (basically duplicates inside one file where it will be actually used) and definitions (actually #68071 is duplicate of "Hide definition" part of this issue) UPD. |
Yes, this isn't a dupe but TBH this is far out of scope because the API doesn't define any semantics on references. Eg the editor doesn't know anything about references, they are just pointers to some line of source. The proposed API for call hierarchies (#70231) will have this knowledge and while the first UX is peek based, there will likely be a viewlet based UX too. |
This issue is being closed to keep the number of issues in our inbox on a manageable level, we are closing issues that are not going to be addressed in the foreseeable future: We look at the number of votes the issue has received and the number of duplicate issues filed. More details here. If you disagree and feel that this issue is crucial: We are happy to listen and to reconsider. If you wonder what we are up to, please see our roadmap and issue reporting guidelines. Thanks for your understanding and happy coding! |
Thank you for explanation. |
Yeah, that would be helpful. Something that enhances this API first: Lines 2718 to 2731 in 4c5705f
Location[] allow to return something comparable to DocumentSymbol . Once we have it in VS Code, we follow up with LSP. It also needs a reasonable default, e.g. all Location-only references are of some default kind and then the UIs can follow...
This a long shot, but don't get me wrong. I do like this feature but it will take a while till we are there |
@jrieken understood. Thank you again! |
It would be great to have an ability to filter found references.
For example:
"Hide imports" that will hide references inside of import statements and direct
require
assignments.At this moment each file usually contain 2+ references and first one if always inside of the import statement, but it is rarely place that devs are looking for.
"Hide definition" there is separate commands "Go to/Peek Definition" and when looking for references-usages won't introduce one more.
In combination with #15 (that recalculated according to filters) this will provide proper count of usages.
The text was updated successfully, but these errors were encountered: