Do not auto save if notebook is untitled #8095
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
rchiodo
approved these changes
Oct 18, 2019
Codecov Report
@@ Coverage Diff @@
## master #8095 +/- ##
=======================================
Coverage 59.22% 59.22%
=======================================
Files 498 498
Lines 22427 22427
Branches 3607 3607
=======================================
Hits 13283 13283
Misses 8309 8309
Partials 835 835
Continue to review full report at Codecov.
|
DonJayamanne
added a commit
that referenced
this pull request
Oct 21, 2019
* Do not auto save if notebook is untitled * Remove redundant code
DonJayamanne
added a commit
that referenced
this pull request
Oct 22, 2019
* Expand variable explorer tests to work for native editor too. (#7699) * Expand variable explorer tests to work for native editor too. * Add comment about async fix for editor open * Review feedback * Allow connection to servers with no token and no password (#7708) * Add capability to support current file path for notebook root (#7724) * Add capability to support current file path for notebook root * Put launch.json back * Use system variables instead of creating separate resolver * Fix mistake in config settings with passing in a uri instead of a string * Make arguments more explicit * Fix IS_WINDOWS to work on unit tests * Better cancellations (#7714) * Variable explorer and mime test fixes for tests (#7740) * Change variable explorer tests to wait for variable explorer complete * Fix native mime test * Review feedback * Try to make 2.7 tests pass * Fix liveshare guest hang issue (#7764) * initial changes * update notebooks on server session change and add test * await correctly * Fix jupyter server startup hang when xeus-cling is installed (#7773) * Make interactive window and native editor take their fontSize and fontFamily from the settings in VS Code. (#7746) * Make interactive window and native editor take their fontSize and fontFamily from the settings in VS Code. We get them in nativeEditor.tsx and in interactivePanel.tsx to apply them as styles, but we also need to send them as props to eventually assign them in the monaco editor. * Moved style from react code to common.css We now get the VS Code styles from the state of the component, instead of computing a style from an element. * Removed getting the font from nativeEditor and interactivePanel, and put it in the mainStateController * Made the interactiveWindow and the nativeEditor react and change their fonts when the user changes them in VS Code, instead of having to reload. * Changed the font props to be an object instead of two separate props to make it easier to maintain. React Context was not used to avoid making the reuse of components more difficult. * Removed some unnecesary passing around of the font prop * removed unnecesary comments * fixed a failing test * Fix a number of perf issues with large notebooks (#7778) * First idea - load cells all at once. Change updates to not update full cell Change global properties to be per cell Change cells to check for updates before rendering * Fix delete to not select at the same time * Make a pending state and a rendered state to use for rendering Some perf fixes for monaco editor. Skip laying out parent etc until necessary * Refactor intellisense to not take so much time on load * More delete and move fixes for intellisense * Fix markdown not switching Fix markdown editor background * Fix some of the time on first render * Add some more unit tests and fix intellisense issue * Add news entry and fix variable explorer open * Remove unnecessary state change * Remove unnecessary merge * Comment change * Postpone widget parent changes * Add new dependency * Update some comments and names to be clearer * Change the default cell marker (#7782) * Change the default cell marker * Fix test failures Fix gather to use new comment Fix formatting on code comments from nls * Fix unit test too * Fix gather to replace #%% * Fix selection and focus not updating when clicking around in a notebook (#7806) * Fix selection and focus not updating when clicking around in a notebook editor. * Also fix add cell to put focus in the new cell * More functional tests for the notebook editor (#7804) * New tests * Add some more functional tests for the notebook editor Make sure when an error occurs on running a cell it's displayed to the user. * Review feedback * Add native command palette cell commands and recontext IW cell commands (#7807) * Tests for keyboard shortcuts in native editor (#7797) * Tests for keyboard shortcuts * Added some more tests * Code review comments * Fix merge issues * Smoke tests does not depend on build stage (#7819) * Smoke tests does not depend on build stage * Faster CI * Tests for undo using keyboard shortcut 'z' (#7824) * Update @types/vscode (#7833) * Update vscode dependency * News entry * Fix focus problems with add new cell buttons (#7827) * Fix focus problems with add new cell buttons * Get rid of unnecessary delay for async callbacks * Use VSC API instead of hardcoding the uri resource vscode-resource (#7835) * Update vscode dependency * News entry * Stop hardcoding 'vscode-resource' and use VSC api * Remove jison dependency (#7863) * Another fix for focus on add (#7883) * Auto save native editor notebook (#7831) * Fix ctrl+s to work in the notebook editor (#7919) * Fix ctrl+s to work in the notebook editor * Fix linting error * More linter fixes * Cherry pick commit b6545e3 * Prevent updates to cell text when stauts of cell has changed (#7973) * Prevent updates to cell text when stauts of cell has changed * Added comments * When automatically opening the `Notebook Editor`, non file schemes (#7921) * Ensure we support live share * Copy *.trie files related to fontkit, as part of webpacking (#7978) * Cherry pick saving fix for ipynb files * new notebook command outside of extension (#7981) * Hide the parameters list when editor doesn't have focus (#7987) * Added comments * Fix max output size to be passed down to rendering (#8017) * Fix intellisense popping up in the wrong spot when first typing in a cell (#8018) * Fix markdown disappearing after editing and hitting the escape key. (#8047) * Fix markdown disappearing after editing and hitting the escape key. * Put back Don's hack * Fixes to allowing users to open a diff view for ipynb files (#8081) * Fixes to allowing users to open a diff view * Updated code review comments * Ensure metadata for notebooks are always present (#8109) * Ensure metadata for notebooks are always present * Let storage run in background * Refactor * Address code review * Do not auto save if notebook is untitled (#8095) * Do not auto save if notebook is untitled * Remove redundant code
kimadeline
added a commit
that referenced
this pull request
Oct 25, 2019
* Ensure datascience smoke tests run only on python3 (#7755) * Add missing PyPI packages to toml (#7749) (#7758) * Fix liveshare guest hang issue (#7764) (#7772) * initial changes * update notebooks on server session change and add test * await correctly * Disable smoke tests against 2.7 (#7774) * Disable flaky smoke tests (#7776) * Disable smoke tests against 2.7 * Restore old smoke tests * Turn on the native editor by default (#7777) * Adding both apache and BSD licenses for `packaging` based on CELA recommendation. (#7763) (#7785) * Interactive Window and Native Editor react to font changes in VS Code settings (#7799) * Make interactive window and native editor take their fontSize and fontFamily from the settings in VS Code. We get them in nativeEditor.tsx and in interactivePanel.tsx to apply them as styles, but we also need to send them as props to eventually assign them in the monaco editor. * Moved style from react code to common.css We now get the VS Code styles from the state of the component, instead of computing a style from an element. * Removed getting the font from nativeEditor and interactivePanel, and put it in the mainStateController * Made the interactiveWindow and the nativeEditor react and change their fonts when the user changes them in VS Code, instead of having to reload. * Changed the font props to be an object instead of two separate props to make it easier to maintain. React Context was not used to avoid making the reuse of components more difficult. * Removed some unnecesary passing around of the font prop * removed unnecesary comments * fixed a failing test * Preparing for release (#7828) * Allow the language server to choose its default caching mode (#7822) * Preparing for release * Update to change logs * Grammar and typo fixes. * Cherry pick inhertEnv and disable debug adapter experiment into release (#8129) * Point release for Extension with data science changes (#8153) * Expand variable explorer tests to work for native editor too. (#7699) * Expand variable explorer tests to work for native editor too. * Add comment about async fix for editor open * Review feedback * Allow connection to servers with no token and no password (#7708) * Add capability to support current file path for notebook root (#7724) * Add capability to support current file path for notebook root * Put launch.json back * Use system variables instead of creating separate resolver * Fix mistake in config settings with passing in a uri instead of a string * Make arguments more explicit * Fix IS_WINDOWS to work on unit tests * Better cancellations (#7714) * Variable explorer and mime test fixes for tests (#7740) * Change variable explorer tests to wait for variable explorer complete * Fix native mime test * Review feedback * Try to make 2.7 tests pass * Fix liveshare guest hang issue (#7764) * initial changes * update notebooks on server session change and add test * await correctly * Fix jupyter server startup hang when xeus-cling is installed (#7773) * Make interactive window and native editor take their fontSize and fontFamily from the settings in VS Code. (#7746) * Make interactive window and native editor take their fontSize and fontFamily from the settings in VS Code. We get them in nativeEditor.tsx and in interactivePanel.tsx to apply them as styles, but we also need to send them as props to eventually assign them in the monaco editor. * Moved style from react code to common.css We now get the VS Code styles from the state of the component, instead of computing a style from an element. * Removed getting the font from nativeEditor and interactivePanel, and put it in the mainStateController * Made the interactiveWindow and the nativeEditor react and change their fonts when the user changes them in VS Code, instead of having to reload. * Changed the font props to be an object instead of two separate props to make it easier to maintain. React Context was not used to avoid making the reuse of components more difficult. * Removed some unnecesary passing around of the font prop * removed unnecesary comments * fixed a failing test * Fix a number of perf issues with large notebooks (#7778) * First idea - load cells all at once. Change updates to not update full cell Change global properties to be per cell Change cells to check for updates before rendering * Fix delete to not select at the same time * Make a pending state and a rendered state to use for rendering Some perf fixes for monaco editor. Skip laying out parent etc until necessary * Refactor intellisense to not take so much time on load * More delete and move fixes for intellisense * Fix markdown not switching Fix markdown editor background * Fix some of the time on first render * Add some more unit tests and fix intellisense issue * Add news entry and fix variable explorer open * Remove unnecessary state change * Remove unnecessary merge * Comment change * Postpone widget parent changes * Add new dependency * Update some comments and names to be clearer * Change the default cell marker (#7782) * Change the default cell marker * Fix test failures Fix gather to use new comment Fix formatting on code comments from nls * Fix unit test too * Fix gather to replace #%% * Fix selection and focus not updating when clicking around in a notebook (#7806) * Fix selection and focus not updating when clicking around in a notebook editor. * Also fix add cell to put focus in the new cell * More functional tests for the notebook editor (#7804) * New tests * Add some more functional tests for the notebook editor Make sure when an error occurs on running a cell it's displayed to the user. * Review feedback * Add native command palette cell commands and recontext IW cell commands (#7807) * Tests for keyboard shortcuts in native editor (#7797) * Tests for keyboard shortcuts * Added some more tests * Code review comments * Fix merge issues * Smoke tests does not depend on build stage (#7819) * Smoke tests does not depend on build stage * Faster CI * Tests for undo using keyboard shortcut 'z' (#7824) * Update @types/vscode (#7833) * Update vscode dependency * News entry * Fix focus problems with add new cell buttons (#7827) * Fix focus problems with add new cell buttons * Get rid of unnecessary delay for async callbacks * Use VSC API instead of hardcoding the uri resource vscode-resource (#7835) * Update vscode dependency * News entry * Stop hardcoding 'vscode-resource' and use VSC api * Remove jison dependency (#7863) * Another fix for focus on add (#7883) * Auto save native editor notebook (#7831) * Fix ctrl+s to work in the notebook editor (#7919) * Fix ctrl+s to work in the notebook editor * Fix linting error * More linter fixes * Cherry pick commit b6545e3 * Prevent updates to cell text when stauts of cell has changed (#7973) * Prevent updates to cell text when stauts of cell has changed * Added comments * When automatically opening the `Notebook Editor`, non file schemes (#7921) * Ensure we support live share * Copy *.trie files related to fontkit, as part of webpacking (#7978) * Cherry pick saving fix for ipynb files * new notebook command outside of extension (#7981) * Hide the parameters list when editor doesn't have focus (#7987) * Added comments * Fix max output size to be passed down to rendering (#8017) * Fix intellisense popping up in the wrong spot when first typing in a cell (#8018) * Fix markdown disappearing after editing and hitting the escape key. (#8047) * Fix markdown disappearing after editing and hitting the escape key. * Put back Don's hack * Fixes to allowing users to open a diff view for ipynb files (#8081) * Fixes to allowing users to open a diff view * Updated code review comments * Ensure metadata for notebooks are always present (#8109) * Ensure metadata for notebooks are always present * Let storage run in background * Refactor * Address code review * Do not auto save if notebook is untitled (#8095) * Do not auto save if notebook is untitled * Remove redundant code * October point release 1 (#8156) * New point release * Mac smoke tests are not completing, but passing * Revert changes * Delete changelog items in point release * Fix native editor merge changes * Undo format on save changes + bad merge * Remove commented out code
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When attempting to auto save untitled notebooks, user will be presented with a save dialog.
This shouldn't be done as
auto saveis not initiated by the user.