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

"Download project" flow is confusing #530

Closed
TwoFX opened this issue Oct 4, 2024 · 2 comments · Fixed by #532
Closed

"Download project" flow is confusing #530

TwoFX opened this issue Oct 4, 2024 · 2 comments · Fixed by #532
Labels
enhancement New feature or request

Comments

@TwoFX
Copy link
Member

TwoFX commented Oct 4, 2024

Description

The "Download project" command shows the following dialog:
image

An easy mistake to make here is to paste in a Git repository URL immediately at this. This leads to a state where hitting Enter does nothing. The correct way to do it would be to first click on "Git repository URL" or hit enter while the text box is empty and then paste in the repository URL and hit Enter. This is easy to get wrong and we should restructure the dialog so that it's less easy to get stuck.

Context

As described in https://leanprover.zulipchat.com/#narrow/stream/113488-general/topic/gitpod.2Fcodespaces.2Flocal.20installation.20for.20Lean.20repo/near/474783261

Steps to Reproduce

  1. Invoke the "Download project" command from the Forall menu
  2. Paste in a Git repo URL immediately
  3. Hit the Enter key -> nothing happens

Expected behavior: Either the flow makes this situation impossible or hitting Enter should do the right thing

Actual behavior: No thing happens and there is no feedback

Versions

0.0.178

Lean (version 4.9.0-nightly-2024-05-23, x86_64-unknown-linux-gnu, commit d984030c6a68, Release)

Linux markus-z16 6.10.11-200.fc40.x86_64 #1 SMP PREEMPT_DYNAMIC Wed Sep 18 21:09:58 UTC 2024 x86_64 GNU/Linux

Impact

Add 👍 to issues you consider important. If others are impacted by this issue, please ask them to add 👍 to it.

@TwoFX TwoFX added the bug Something isn't working label Oct 4, 2024
@mhuisi
Copy link
Collaborator

mhuisi commented Oct 4, 2024

Sorry, but this is not possible with VS Code's API. We can't remove the search bar and we can't make the search bar select "Git repository URL" for an arbitrary search string. You can find the whole quick pick API provided by VS Code here. There is also no other VS Code UI component we could use here.

The only options I see is that we can either have a template selector of this form here, we have no template selector at all or we re-implement the entirety of all VS Code UI components we use in commands for dialogs in separate webviews.

@mhuisi mhuisi added enhancement New feature or request and removed bug Something isn't working labels Oct 4, 2024
@mhuisi
Copy link
Collaborator

mhuisi commented Oct 4, 2024

@bollu made me aware of the fact that there's some more API at window.createQuickPick that we could use to improve the UI of the quick pick mechanism, so I think that this is feasible after all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants