Skip to content

references-view: removeReferenceItem hotkey implementation #237486

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

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

Conversation

Clam-
Copy link

@Clam- Clam- commented Jan 8, 2025

Resolves #234017 and #200465.

Previously removeReferenceItem did not function when utilized via hotkey. This is because the function called by the hotkey does not have the "item" to remove.
This is resolved by iterating over the current selection and calling remove on those items in the case where no item is provided.

This can be tested by:

  • Assigning a hotkey to removeReferenceItem
  • Right clicking on a symbol and selecting "Find All References"
  • Pressing the hotkey

@Clam-
Copy link
Author

Clam- commented Jan 8, 2025

@microsoft-github-policy-service agree

@ckx
Copy link

ckx commented Feb 19, 2025

@jrieken Is there a time frame we can expect this PR to be reviewed within?

@Clam-
Copy link
Author

Clam- commented Apr 10, 2025

@jrieken hiya. Is there something blocking this? Please let me know if I need to make any adjustments or do anything else.
Thank you.

@ckx
Copy link

ckx commented Jun 21, 2025

It has been over half a year since this PR was created. Is there any chance of it being reviewed?

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.

Some references.view shortcuts not working
3 participants