-
Notifications
You must be signed in to change notification settings - Fork 907
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
Enable hot exit for saved files #469
Conversation
@@ -267,7 +267,7 @@ configurationRegistry.registerConfiguration({ | |||
'files.hotExit': { | |||
'type': 'string', | |||
'enum': [HotExitConfiguration.OFF, HotExitConfiguration.ON_EXIT, HotExitConfiguration.ON_EXIT_AND_WINDOW_CLOSE], | |||
'default': HotExitConfiguration.ON_EXIT, | |||
'default': HotExitConfiguration.OFF, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just merged your change to force off HotExit. you probably should merge that change and set it to read from the config and default to OFF.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's test this some more tomorrow and decide if this is something we can take for Jan release.
8d0e483
to
fa487d2
Compare
In 0.25.3, it looks like HotExit is disabled (receive save dialog when exiting and files are gone after reopening). Is this the intended behavior for now? |
@mrlife Hot exit is disabled by default in this build, but can be enabled in settings (ctrl + ,) by changing "files.hotExit" to "onExit" or "onExitAndWindowClose". We'll make sure to document these steps for the upcoming release. If that doesn't work for you, please let me know. Thanks! |
* fix file browser service bug * modify filebrowser service * change filebrowser contract * fix contract * fix file browser add/expand event contracts * remove commented code
This will fix #463 by changing how we serialize and deserialize editor windows to always serialize VS Code's version of the editor, then convert it to a
SqlInput
editor upon deserialization if needed.This is still a work in progress and needs some testing, but opening the PR now in case others are interested in looking at the code change.