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 copy feature to call hierarchy #223819

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

nagi1999a
Copy link

Implemented #162474.

There all five context menu items added:

  1. Copy
    • Copy the name of the function
       setCallsDirection
      
  2. Copy Hierarchy
    • Copy the hierarchy with symbol type and location
      setCallsDirection (Function @ src/calls/index.ts)
        > register (Function @ src/calls/index.ts)
          > activate (Function @ src/extension.ts)
      
  3. Copy Hierarchy (Reversed)
    • Copy the hierarchy with symbol type and location in a reversed direction
      activate (Function @ src/extension.ts)
        > register (Function @ src/calls/index.ts)
          > setCallsDirection (Function @ src/calls/index.ts)
      
  4. Copy Hierarchy (Short)
    • Copy the hierarchy with the name only
      setCallsDirection
        > register
          > activate
      
  5. Copy Hierarchy (Reversed, Short)
    • Copy the hierarchy with the name only in a reversed direction
      activate
        > register
          > setCallsDirection
      

Limitation:
The current implementation only considers the ancestors of the selection item.

@nagi1999a
Copy link
Author

@microsoft-github-policy-service agree

@nagi1999a
Copy link
Author

Sorry for the force git push since some errors happened during the upload process.

@JulienYo
Copy link

Hello @jrieken @mjbvz, we are very interested in this feature for our plugin, any chance this review may be done soon ?

Thanks.

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

Successfully merging this pull request may close these issues.

3 participants