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

CHE-3570. Add ability to turn on/off autosave mode for editor content #5170

Merged
merged 1 commit into from
May 31, 2017

Conversation

RomanNikitenko
Copy link
Member

Signed-off-by: Roman Nikitenko rnikitenko@codenvy.com

What does this PR do?

Add option 'Enable Autosave' to editor preferences.
Provide:

  • sync opened file content with working copy on server side when the file has unsaved data.
  • update editor content for 'Split' actions when Autosave mode is disabled.
  • processing 'Reconcile' operations for java files and pom.xml files from working copy on server side

What issues does this PR fix or reference?

#3570

Changelog

Add ability to turn on/off autosave mode for editor content

autosave

@codenvy-ci
Copy link

Build # 2651 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/2651/ to view the results.

@codenvy-ci
Copy link

@davidwindell
Copy link
Contributor

Is there a Ctrl + S save command when autosave is turned off?

@RomanNikitenko
Copy link
Member Author

@davidwindell Yes, there is Ctrl+S hot key for save a content of current editor
hot-key

Signed-off-by: Roman Nikitenko <rnikitenko@codenvy.com>
@codenvy-ci
Copy link

@RomanNikitenko RomanNikitenko merged commit 1c7818f into master May 31, 2017
@RomanNikitenko RomanNikitenko deleted the CHE-3570 branch May 31, 2017 13:29
@RomanNikitenko RomanNikitenko added this to the 5.12.0 milestone May 31, 2017
@slemeur slemeur added the kind/enhancement A feature request - must adhere to the feature request template. label Jun 15, 2017
@slemeur
Copy link
Contributor

slemeur commented Jun 15, 2017

@vparfonov + @RomanNikitenko : This is a new capability and you need to request PMs to review the PR before merging. That PR actually needed documentation and release note information.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A feature request - must adhere to the feature request template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants