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

Display welcome page on install #128

Closed
wants to merge 1 commit into from
Closed

Display welcome page on install #128

wants to merge 1 commit into from

Conversation

xorye
Copy link
Contributor

@xorye xorye commented Oct 28, 2019

Fixes #127

This PR implements a creative/hacky fix to display the welcome page on extension install. (ie, displays the welcome page when vscode-quarkus is installed)

The fix was to have vscode-quarkus activate on startup, show the welcome page and have vscode-quarkus manually edit the package.json file so that the startup activation event is removed.

Hi @jrieken, what do you think about this fix?

I have created a feature request for an "onExtensionInstalled" activation event to the VS Code GitHub page here: microsoft/vscode#83451

@jrieken
Copy link

jrieken commented Oct 28, 2019

Hi @jrieken, what do you think about this fix?

I think that's a bad idea and you might trigger an "installation is corrupt" warning when changing your extension after install. Generally, we are looking into a streamlined way that allows extensions to show a welcome or getting started page and my recommendation is to join us on designing that. /cc @eamodio

@fbricon
Copy link
Collaborator

fbricon commented Oct 28, 2019

damn that was a cool trick ;-)

@jrieken is there an existing issue we could start watching about improving the 3rd party welcome page situation?

@fbricon fbricon closed this Oct 28, 2019
@fbricon fbricon added the invalid This doesn't seem right label Oct 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Welcome page appears too late
4 participants