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

Add CTRL+Click "Open Call Hierarchy" Action on classes #1231

Closed
flori-schwa opened this issue Mar 5, 2024 · 4 comments · Fixed by #1367 or eclipse-platform/www.eclipse.org-eclipse#167
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@flori-schwa
Copy link

flori-schwa commented Mar 5, 2024

In #575 new functionality was implemented, that adds a quick way of opening the call hierarchy on methods and fields via CTRL+Click. However this functionality is missing on classes, where the Call hierarchy shows the call hierarchy of all constructors (or the default constructor).

CTRL+Hover on Constructor: The "Open Call Hierarchy" Action is properly displayed
grafik

CTRL+Hover on Class Declaration: The "Open Call Hierarchy" Action is missing
grafik

Call Hierarchy View when opened via Right Click -> Open Call Hierarchy on Class Declaration: In my opinion, this view should also be openable using CTRL+Click "Open Call Hierarchy" on classes.
grafik

@iloveeclipse iloveeclipse added the enhancement New feature or request label Mar 5, 2024
@iloveeclipse iloveeclipse changed the title CTRL+Click "Open Call Hierarchy" Action missing on classes Add CTRL+Click "Open Call Hierarchy" Action on classes Mar 5, 2024
@iloveeclipse
Copy link
Member

Contributions are welcome.

RedeemerSK added a commit to RedeemerSK/eclipse.jdt.ui that referenced this issue Apr 24, 2024
@RedeemerSK
Copy link
Contributor

#1367

RedeemerSK added a commit to RedeemerSK/eclipse.jdt.ui that referenced this issue Apr 24, 2024
@iloveeclipse iloveeclipse added this to the 4.32 M3 milestone May 2, 2024
@iloveeclipse
Copy link
Member

@RedeemerSK : thanks for the patch!

Could you please create a PR with a N&N entry for this, repository is https://github.com/eclipse-platform/www.eclipse.org-eclipse and an example of such entry is eclipse-platform/www.eclipse.org-eclipse#163

@RedeemerSK
Copy link
Contributor

@RedeemerSK : thanks for the patch!

Could you please create a PR with a N&N entry for this, repository is https://github.com/eclipse-platform/www.eclipse.org-eclipse and an example of such entry is eclipse-platform/www.eclipse.org-eclipse#163

Sorry for delay.
eclipse-platform/www.eclipse.org-eclipse#167

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment