commit 5e7a61982b00275d28b0e10b48d2abd81f5b0e09
author Michael Zhou <zhoumotongxue008@gmail.com> Fri Mar 11 00:38:29 2016 -0500
committer Michael Zhou <zhoumotongxue008@gmail.com> Fri Mar 11 01:19:37 2016 -0500
tree 73a69a0af19461f3ac2c22ed1e6ca200f296cf7e
parent 807dcd3d35f0cc7df2ff7caf576de3e08cd9d4ea

Fix emailStrategy ListBox not displayed on MyPreferencesScreen

Commit 5d8f62c938bb9e1e7d493ffe56159cc3f82fd098 left out a "row++",
causing the "signed-off-by footer" CheckBox to replace the
emailStrategy ListBox instead of appearing on the next row.

The current preferences screen looks a bit messy. I'm reordering the
widgets in a follow-up change.

Change-Id: I7bff76be8262fa842e7bb4355b81b230cdb2742d

gerrit-gwtui/src/main/java/com/google/gerrit/client/account/MyPreferencesScreen.java

diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/account/MyPreferencesScreen.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/account/MyPreferencesScreen.java
index f605abf..bcbdd6a 100644
--- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/account/MyPreferencesScreen.java
+++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/account/MyPreferencesScreen.java
@@ -199,6 +199,7 @@
 
     formGrid.setText(row, labelIdx, Util.C.emailFieldLabel());
     formGrid.setWidget(row, fieldIdx, emailStrategy);
+    row++;
 
     formGrid.setText(row, labelIdx, "");
     formGrid.setWidget(row, fieldIdx, signedOffBy);