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

API: Add enablement to welcome view buttons #106487

Closed
gjsjohnmurray opened this issue Sep 11, 2020 · 5 comments · Fixed by #107705
Closed

API: Add enablement to welcome view buttons #106487

gjsjohnmurray opened this issue Sep 11, 2020 · 5 comments · Fixed by #107705
Assignees
Labels
feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders on-release-notes Issue/pull request mentioned in release notes on-testplan workbench-views Workbench view issues
Milestone

Comments

@gjsjohnmurray
Copy link
Contributor

Below is a recording of a Control-R reload when no workspace is open and one non-builtin extension is contributing to to the Welcome view of Explorer.

junk

Observe how we start with Explorer's usual "NO FOLDER OPEN" view. Next my extension contributes its text and buttons. Finally the built-in git extension contributes text and a button. Since built-ins get higher priority the content from my extension jumps downward, perhaps while I'm reading it or trying to click one of its actions (link, button).

There's no progress indicator when the content is being fetched and assembled, and it's hard to know how long to wait before concluding that no other extensions are going to contribute.

VS Code version: Code 1.49.0 (e790b93, 2020-09-10T13:22:08.892Z)
OS version: Windows_NT x64 10.0.18363

@gjsjohnmurray
Copy link
Contributor Author

gjsjohnmurray commented Sep 11, 2020

Similarly on the Source Control view, it starts with a message saying "No source control providers registered" and stays this way long enough for me to worry that something is broken or missing. Eventually the contribution from the git extension arrives.

If a progress indicator isn't added, maybe at least the initial message could say something like "Please wait while we look for source control providers". Then if nothing contributes (e.g. the git extension has been disabled,) it could then display "No source control providers registered".

@joaomoreno
Copy link
Member

This will be very hard to achieve. The problem here isn't actually the extension point, for which we could definitely add a progress indicator, but the activation of the git extension. That git section won't appear until the git extension is activated and all the potential repo locations have been probed... We simply can't add a progress indicator for that in the core unless we add some more API.

@joaomoreno
Copy link
Member

Similarly on the Source Control view, it starts with a message saying "No source control providers registered" and stays this way long enough for me to worry that something is broken or missing. Eventually the contribution from the git extension arrives.

On this, we could potentially add a Git extension is activating... message.

@joaomoreno joaomoreno added the under-discussion Issue is under discussion for relevance, priority, approach label Sep 15, 2020
@gjsjohnmurray
Copy link
Contributor Author

On this, we could potentially add a Git extension is activating... message.

That might help.

@joaomoreno
Copy link
Member

Closing this until we get actual user feedback that it's needed, sorry @gjsjohnmurray!

@joaomoreno joaomoreno added feature-request Request for new features or functionality workbench-views Workbench view issues and removed under-discussion Issue is under discussion for relevance, priority, approach labels Nov 12, 2020
@joaomoreno joaomoreno added this to the Backlog milestone Nov 12, 2020
@joaomoreno joaomoreno reopened this Nov 12, 2020
@joaomoreno joaomoreno modified the milestones: Backlog, November 2020 Nov 12, 2020
@joaomoreno joaomoreno changed the title No progress indicator when extensions are contributing Welcome views to Explorer API: Add enablement to welcome view buttons Nov 23, 2020
@joaomoreno joaomoreno added the on-release-notes Issue/pull request mentioned in release notes label Dec 7, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Jan 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders on-release-notes Issue/pull request mentioned in release notes on-testplan workbench-views Workbench view issues
Projects
None yet
4 participants
@joaomoreno @gjsjohnmurray @chrmarti and others