diff --git a/Documentation/images/user-review-ui-change-screen-replying.png b/Documentation/images/user-review-ui-change-screen-replying.png index 0fd5e80981..0ae85ab0f4 100644 Binary files a/Documentation/images/user-review-ui-change-screen-replying.png and b/Documentation/images/user-review-ui-change-screen-replying.png differ diff --git a/Documentation/user-review-ui.txt b/Documentation/user-review-ui.txt index 81ceb06793..1c3967d6d0 100644 --- a/Documentation/user-review-ui.txt +++ b/Documentation/user-review-ui.txt @@ -602,11 +602,6 @@ comment needs to be edited. The `Post` button publishes the comments and the votes. -The `send email` checkbox controls whether the reply should trigger -email notifications for other users. Deselecting the checkbox means -that there will be no email notification about the change update to the -change author, the reviewers or any other user. - image::images/user-review-ui-change-screen-replying.png[width=800, link="images/user-review-ui-change-screen-replying.png"] If a user can approve a label that is still required, a quick approve diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/ReplyBox.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/ReplyBox.java index 5b8135c492..ddd650e8fb 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/ReplyBox.java +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/ReplyBox.java @@ -99,7 +99,6 @@ class ReplyBox extends Composite { @UiField Element labelsParent; @UiField Grid labelsTable; @UiField Button post; - @UiField CheckBox email; @UiField Button cancel; @UiField ScrollPanel commentsPanel; @UiField FlowPanel comments; @@ -188,15 +187,6 @@ class ReplyBox extends Composite { } } - @UiHandler("email") - void onEmail(ValueChangeEvent e) { - if (e.getValue()) { - in.notify(ReviewInput.NotifyHandling.ALL); - } else { - in.notify(ReviewInput.NotifyHandling.NONE); - } - } - @UiHandler("post") void onPost(ClickEvent e) { postReview(); diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/ReplyBox.ui.xml b/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/ReplyBox.ui.xml index 1b122c5179..3c927fc0d3 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/ReplyBox.ui.xml +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/ReplyBox.ui.xml @@ -68,7 +68,7 @@ limitations under the License. styleName='{res.style.button}'>
Post
- and send email +