Skip to content
This repository was archived by the owner on Mar 18, 2024. It is now read-only.

Conversation

@sqs
Copy link
Member

@sqs sqs commented Jan 4, 2019

@sqs sqs requested a review from lguychard January 4, 2019 07:12
@sqs sqs force-pushed the fuzzy-locations branch from f3dc835 to 181f77c Compare January 4, 2019 07:13
sqs added a commit to sourcegraph/sourcegraph-public-snapshot that referenced this pull request Jan 6, 2019
This lets extensions' location providers (definition/reference/etc. providers) associate context data with each location, and also contribute actions that are shown selectively in the file title for location matches shown in the panel.

Together, these new features allow the basic-code-intel extension to add "Fuzzy" badges to defs/refs in the panel. This fixes sourcegraph/sourcegraph#1174.

It would also enable things like:

- Showing a "source" badge on cross-repository reference results explaining how the reference was found (eg by looking up dependents on npm)
- Showing the type of reference (eg denoting some references as "Assignments", some as "Calls", some as "References", etc.)
- Adding an action to "hide" or "always ignore" the reference
- Adding an action to add the match to a saved list
- Adding an action for users to say "This was useful" or "This was not useful" (eg for a code examples extension)

See sourcegraph/code-intel-extensions#10 for an example of how extensions can use this new API.
sqs added a commit to sourcegraph/sourcegraph-public-snapshot that referenced this pull request Jan 8, 2019
This lets extensions' location providers (definition/reference/etc. providers) associate context data with each location, and also contribute actions that are shown selectively in the file title for location matches shown in the panel.

Together, these new features allow the basic-code-intel extension to add "Fuzzy" badges to defs/refs in the panel. This fixes sourcegraph/sourcegraph#1174.

It would also enable things like:

- Showing a "source" badge on cross-repository reference results explaining how the reference was found (eg by looking up dependents on npm)
- Showing the type of reference (eg denoting some references as "Assignments", some as "Calls", some as "References", etc.)
- Adding an action to "hide" or "always ignore" the reference
- Adding an action to add the match to a saved list
- Adding an action for users to say "This was useful" or "This was not useful" (eg for a code examples extension)

See sourcegraph/code-intel-extensions#10 for an example of how extensions can use this new API.
@chrismwendt
Copy link
Contributor

Closing in favor of #41

@chrismwendt chrismwendt closed this Mar 8, 2019
@efritz efritz deleted the fuzzy-locations branch February 3, 2020 14:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants