-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Launchpad: Search for another PR #3543
Comments
Question. In the old code I see how the |
@sergeibbb VSCode indeed internally filters the quickpick items to only those that match your search (though I'm not sure exactly what parts of the quickpick items it uses to match, other than their main label). There is an |
I have 3 questions for you. The first is a noob one. What does the returning boolean from the event handler means in a QuickPickStep?
The next question was discussed on the daily meeting today. So, no need to answer. We also discussed the UX flows that should be used in this task. I'll put it in the description soon.
Yes. I see it. |
Add ability to search for a PR not in the Launchpad list. UX needs to be talked through.
Normally our algorithm provides the Launchpad with some subset of all PRs. So, the full list of PRs accessible to the user is wider. The current search looks only for PRs selected for the Launchpad and we want user to be able to search among all other available PRs.
The current idea, but needs discussion and may be changed:
https://github.com/gitkraken/vscode-gitlens/pull/3500
. Or if that fails, we could just do a PR search with the integration using the user's search query.Notes
integration.getPullRequest
for the PR search when the user uses PR url, andintegration.searchPullRequests
for more general search, but the exact function/query may change depending on final design.integration.getPullRequest
orintegration.searchPullRequests
, they must be implemented for GitLab since Launchpad supports GitLab as well (currently it is only implemented for GitHub). This might be a good opportunity to add this functionality to the shared provider library, but for now we can implement them locally for GitLab as needed.Specs
Possible UX solutions
"Gradual discovering" of the feature.
General idea is the following:
Search and show the result in a new section
If so, and the goal here is to allow users to easily find any PR they have access to, then we could insert a new section at the bottom of the Launchpad that returns all matches when someone has an active search, without them needing to select another option.
The text was updated successfully, but these errors were encountered: