Skip to content

Also do attach for goto symbols#304841

Merged
TylerLeonhardt merged 2 commits intomainfrom
tyler/useful-stingray
Mar 25, 2026
Merged

Also do attach for goto symbols#304841
TylerLeonhardt merged 2 commits intomainfrom
tyler/useful-stingray

Conversation

@TylerLeonhardt
Copy link
Member

a missed part of #304521

this adds the same handling of shift+enter for goto symbols just like anything & workspace symbols quick access.

a missed part of  #304521

this adds the same handling of shift+enter for goto symbols just like anything & workspace symbols quick access.
Copilot AI review requested due to automatic review settings March 25, 2026 18:00
@TylerLeonhardt TylerLeonhardt enabled auto-merge (squash) March 25, 2026 18:00
@TylerLeonhardt TylerLeonhardt self-assigned this Mar 25, 2026
@vs-code-engineering vs-code-engineering bot added this to the 1.114.0 milestone Mar 25, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aligns Go to Symbol in Editor Quick Access (@) with the “attach via Shift+Enter” behavior already supported in other Quick Access providers (e.g. Anything / Workspace Symbols), enabling symbol picks to be attached (e.g. to chat context) instead of navigating.

Changes:

  • Extend IGotoSymbolQuickPickItem.attach to receive modifier state and the accept event.
  • Intercept onDidAccept to delegate to attach when Shift is held, skipping navigation.

@TylerLeonhardt TylerLeonhardt merged commit d9bb1c3 into main Mar 25, 2026
47 of 51 checks passed
@TylerLeonhardt TylerLeonhardt deleted the tyler/useful-stingray branch March 25, 2026 23:05
justschen pushed a commit that referenced this pull request Mar 26, 2026
a missed part of  #304521

this adds the same handling of shift+enter for goto symbols just like anything & workspace symbols quick access.
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