-
Notifications
You must be signed in to change notification settings - Fork 248
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
Why is LiveShare *
activated?
#4470
Comments
This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
Still would love to know why. This can help us improve the activation events of vscode. |
Hey @TylerLeonhardt, I started looking into this issue. We are open to exploring activating Live Share upon a different activation event. However, Live Share has some scenarios where we need to reload VS Code and automatically reconnect to a Live Share session. One example of these scenarios is when a user joins a session as an anonymous user and then signs in during the session. To do this, we reload VS Code and upon reload, automatically rejoin them to the session as a signed-in user. For this scenario, we need to immediately activate Live Share, even if the user takes no action. Is there a way we could change our activation event for the majority of use cases while still keeping this behavior? Also, is this a problem for vscode.dev, VS Code, or both? |
@alyssajotice correct me if I'm wrong but wouldn't the activation event that you already have listed called
Both. The Activation events work the same way in this case. |
Thanks for the suggestion. Let me give it a try and I'll make sure it works. |
The only necessity for Live Share activating on |
This would be my preference. When I use live share, I always am chatting with someone on Teams/Slack first and then they give me the LiveShare link that I open. I don't think I've ever invited someone out of the blue using LiveShare's tree view. With that said, activation events are static so you could use a separate extension that maybe has this "always on" functionality.
|
Any updates on this one? |
@TylerLeonhardt: I was taking a look at this one. As a quick test I tried replacing "*" with: activationEvents": [ I noticed that the live share icon does not show up in the Activity Bar or Status Bar though until it is activated by invoking a liveshare command from the command palette. Is there a way to get these to show up without activating? Then if the user clicks on the icon in the Activity Bar, or invokes the share command from the Status Bar button, we can activate via the events listed above? Before Activation: After Activation: |
Linking to microsoft/vscode#167874 |
Moved away from |
Resolving this one as we are no longer * activated. We can track further optimizations with #167874 |
Describe the bug
When you are
*
activated, you compete with vscode for start up time and as a result, you hurt vscode's startup time in the process.It's better to be reactive and only activate when the user interacts with your extension:
To Reproduce
Steps to reproduce the behavior:
Expected behavior
It shouldn't activate until needed.
Additional context
Activation events for your reference (even though I know you know where these are)
https://code.visualstudio.com/api/references/activation-events
The text was updated successfully, but these errors were encountered: