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-7880: Save latest commit message before showing Git Commit dialog #8004

Merged
merged 6 commits into from
Dec 26, 2017

Conversation

vinokurig
Copy link
Contributor

What does this PR do?

Save latest commit message before showing Git Commit dialog

What issues does this PR fix or reference?

#7880

Latest commit message sets without delay when amend option is switched on in Git Commit dialog

Release Notes

N/A

Docs PR

N/A

@vinokurig vinokurig added the kind/enhancement A feature request - must adhere to the feature request template. label Dec 21, 2017
@vinokurig vinokurig added the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Dec 21, 2017
@@ -137,6 +142,9 @@ public void showDialog(Project project) {
newFiles.addAll(status.getUntracked());
show(newFiles.stream().collect(joining("\nA\t", "A\t", "")));
});
} else {
view.setMessage("");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess it isn't necessary.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

notificationManager.notify(constant.logFailed(), FAIL, NOT_EMERGE_MODE);
}
});
if (latestCommitMessage == null) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just set focus to commit message text box

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added setting focus to message field on clicking the amend checkbox.

@vinokurig
Copy link
Contributor Author

ci-test

@codenvy-ci
Copy link

@vinokurig
Copy link
Contributor Author

ci-test

@codenvy-ci
Copy link

@vinokurig vinokurig changed the title CHE-7800: Save latest commit message before showing Git Commit dialog CHE-7880: Save latest commit message before showing Git Commit dialog Dec 25, 2017
@vinokurig
Copy link
Contributor Author

ci-test

@codenvy-ci
Copy link

@vinokurig vinokurig merged commit c43bb02 into master Dec 26, 2017
@vinokurig vinokurig deleted the CHE-7880 branch December 26, 2017 08:05
@vinokurig vinokurig removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Dec 26, 2017
@vinokurig vinokurig added this to the 6.0.0-M4 milestone Dec 26, 2017
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