diff --git a/src/main/java/com/google/gerrit/client/patches/AbstractPatchContentTable.java b/src/main/java/com/google/gerrit/client/patches/AbstractPatchContentTable.java index 90fb66e68e..f9e109791b 100644 --- a/src/main/java/com/google/gerrit/client/patches/AbstractPatchContentTable.java +++ b/src/main/java/com/google/gerrit/client/patches/AbstractPatchContentTable.java @@ -107,6 +107,17 @@ public abstract class AbstractPatchContentTable extends FancyFlexTable { case KeyboardListener.KEY_UP: case KeyboardListener.KEY_DOWN: return false; + + case 'c': + case 'r': + for (int row = getCurrentRow(); 0 <= row; row--) { + final Object item = getRowItem(row); + if (!(item instanceof CommentList) && item != null) { + onOpenItem(item); + break; + } + } + return true; } } return super.onKeyPress(keyCode, modifiers); diff --git a/src/main/java/com/google/gerrit/client/ui/FancyFlexTable.java b/src/main/java/com/google/gerrit/client/ui/FancyFlexTable.java index f3802a9018..14c075fc48 100644 --- a/src/main/java/com/google/gerrit/client/ui/FancyFlexTable.java +++ b/src/main/java/com/google/gerrit/client/ui/FancyFlexTable.java @@ -155,7 +155,7 @@ public abstract class FancyFlexTable extends Composite implements } } - private void onOpen() { + protected void onOpen() { if (0 <= currentRow && currentRow < table.getRowCount()) { final RowItem item = getRowItem(currentRow); if (item != null) {