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

Trigger file saving operation on editor closing only when file has changes #4393

Merged
merged 1 commit into from
Mar 13, 2017

Conversation

vzhukovs
Copy link
Contributor

What does this PR do?

This changes proposal disable automatically file saving on editor closing.

The previous version doesn't care about whether file changed or not and automatically fired file saving. New behavior takes care about current opened file, and if file is dirty then fire file saving.

Signed-off-by: Vladyslav Zhukovskii vzhukovskii@codenvy.com

What issues does this PR fix or reference?

#4076

Changelog

Trigger file saving operation on editor closing only when file has changes

Release Notes

Trigger file saving operation on editor closing only when file has changes

Docs PR

N/A

…anges

Signed-off-by: Vladyslav Zhukovskii <vzhukovskii@codenvy.com>
@vzhukovs vzhukovs requested a review from vparfonov March 10, 2017 15:13
@vzhukovs vzhukovs self-assigned this Mar 10, 2017
@vzhukovs vzhukovs added the kind/bug Outline of a bug - must adhere to the bug report template. label Mar 10, 2017
@vzhukovs
Copy link
Contributor Author

Selenium testing wasn't found any regression

@codenvy-ci
Copy link

Build finished.
Build success. $BUILD_URL

@codenvy-ci
Copy link

@vparfonov vparfonov added this to the 5.5.0 milestone Mar 13, 2017
@vzhukovs vzhukovs merged commit 194f044 into master Mar 13, 2017
@vzhukovs vzhukovs deleted the che#4076 branch March 13, 2017 08:23
@JamesDrummond JamesDrummond mentioned this pull request Mar 17, 2017
9 tasks
@slemeur slemeur added kind/enhancement A feature request - must adhere to the feature request template. and removed kind/bug Outline of a bug - must adhere to the bug report template. labels Mar 30, 2017
JPinkney pushed a commit to JPinkney/che that referenced this pull request Aug 17, 2017
…anges (eclipse-che#4393)

Signed-off-by: Vladyslav Zhukovskii <vzhukovskii@codenvy.com>
This pull request was closed.
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.

4 participants