-
Notifications
You must be signed in to change notification settings - Fork 30.7k
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
Support synchronising global state #86679
Comments
@dbaeumer How feasible it is to move locale configuration to settings? |
The biggest problem moving it to settings is performance. It is its own file since we need to read that file early on in the startup process and we didn't want to read large files and parse them. So would we be able to sync it separately ? |
You are right and it might impact the performance. If so, lets not move it to settings and I can sync locale separately. |
Thanks! |
There are two issues I think are relevant to this: 1. Sessions across machines: You might want to think about how you want to handle session restores across different machines that doesn't have access to the same files, perhaps by design (e.g. work machine, home machine). I encountered this issue a while back in the portable version of VSCode, which actually prompted me to stop using the portable version: #81340 2. Settings across machines: This is also part of a bigger problem of
|
Can you please elaborate more about the issue? |
I'm not 100% sure how this is relevant to this issue since not many details are provided. Perhap you could elaborate beyond the title "Support synchronising global state" what this means? I just meant everything I said in #81340 - but I can re-iterate it here if necessary. If you're looking for VSCode to restore editor session across different machines, as the portable version inherently does, you're going to have potentially not access to the same files. So you get errors that you cannot open file X, Y and Z when you open VSCode on another machine. For example, I'm on machine A and have files X, Y and Z open. |
I see your use case. We have plans to sync open editors state but definitely not in scope of this issue. |
Fair enough 🙂So though the title of this issues says "Support synchronising global state", your inital post seem to suggest this is just about the locale setting for now (e.g. country + language)? |
Yeah currently we only support syncing display language as part of global state. Planning to extend it to more global state like activity bar icons, views visibilities etc., |
Makes sense to add the basic functionality first 👍 I hope that somewhere down the line that grouping of session will be considered at some point (Group A will share these states, Group B will share these states, all groups will share these states etc.). |
Syncing global state (core) is supported now. Next step is to let state owners to opt in to sync the state - #93332 |
@sandy081 how will extensions opt-in or out of this? |
Currently it is only supported for core. Planning to extend this to extensions in next milestone. |
Support synchronising global state
The text was updated successfully, but these errors were encountered: