diff options
author | Ismo Haataja <ismo.haataja@digia.com> | 2014-09-22 15:08:29 +0300 |
---|---|---|
committer | Ismo Haataja <ismo.haataja@digia.com> | 2014-10-15 09:16:23 +0200 |
commit | d124a6d287ef836b3290d92baa2f1e14b18a36bd (patch) | |
tree | 0bb605b560340691c21c1b74730630514b9acf2c | |
parent | 958f2a44a03e215d78667c28296b0c22b3df9d07 (diff) |
No initial focus to Cover Message editor in One Page Review
Setting initial focus to Cover Message editor made no sense because
it is in bottom of page, so usually not even visible and not logical
place to start review from. This also disabled keyboard navigation keys.
Now focus is not explicitly set and keyboard navigation works.
Task-number: QTQAINFRA-883
Change-Id: I5b80f7236d4863bb36a493fa43f2dcc58b2ffd43
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>
Reviewed-by: Ismo Haataja <ismo.haataja@digia.com>
-rw-r--r-- | gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/AllInOnePatchScreen.java | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/AllInOnePatchScreen.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/AllInOnePatchScreen.java index fcbab486c2..d2d3c3c6ac 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/AllInOnePatchScreen.java +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/AllInOnePatchScreen.java @@ -442,11 +442,6 @@ public class AllInOnePatchScreen extends AbstractPatchScreen implements submit.setEnabled(result.getChange().isMergeable()); } } - - @Override - protected void postDisplay() { - message.setFocus(true); - } })); } |