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

call hierarchy not available in TypeScript file #1590

Open
mkq opened this issue Jun 3, 2024 · 4 comments
Open

call hierarchy not available in TypeScript file #1590

mkq opened this issue Jun 3, 2024 · 4 comments

Comments

@mkq
Copy link

mkq commented Jun 3, 2024

(Moved from (https://bugs.eclipse.org/bugs/show_bug.cgi?id=583302):)

In a TypeScript file (opened with Generic Text Editor by default) in the context menu on a function declaration, "Open Call Hierarchy" is disabled. Could you please make it work?

(The required information seems to be available: Ctrl+Shift+G "Find References" finds callers; and F3 "Open Declaration" on a caller navigates to the called function.)

I love the call hierarchy (in Java). It's so much simpler than emulating it with consecutive "Find References" where I get lost quickly.

I used a new workspace and a fresh installation of eclipse-jee-2024-03-R-win32-x86_64.zip AKA:
Eclipse IDE for Enterprise Java and Web Developers (includes Incubating components)
Version: 2024-03 (4.31.0)
Build id: 20240307-1437

Thanks!

@mickaelistria
Copy link
Contributor

I'm not much familiar with "Call hierarchy" to be honest.
But if you think it can be implemented with a combination of standard LSP features (find-references + open declaration), then it could be a good addition to LSP4E project directly so that would work for all languages that are supported by a language server.
What do you think?

@mkq
Copy link
Author

mkq commented Jun 10, 2024

Sorry. I didn't want to claim that I know how to implement that. I don't know anything about Eclipse development.

I just meant - purely from the user's perspective - it looks like the callers information is available, so it shouldn't be too much effort to display it as a call hierarchy tree.

@mickaelistria
Copy link
Contributor

it shouldn't be too much effort to display it as a call hierarchy tree.

Probably not. But at the moment, I don't think this can become a priority of the maintainers. However, if you're willing to contribute it, most of us would be happy to assist you in doing so and to review and merge good code.

@mkq
Copy link
Author

mkq commented Jun 10, 2024

That's a pity.

I consider call hierarchy a basic IDE feature. I always find it baffling when I see a colleague use multiple "find references" and mental backtracking, while in a call hierarchy, you can immediately see where to continue your search (i. e. expand not yet expanded nodes). I will probably continue to use VSCode for TypeScript (which is another pity, because I hate IDEs / code editors with such a limited view / perspective management).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants